Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sriram - added new endpoint TimeEntry/users/totalHours #1209

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

sri0606
Copy link

@sri0606 sri0606 commented Jan 27, 2025

Description

image

Related PRS (if any):

This backend PR is related to the frontend PR
To test this backend PR you need to checkout the #XXX frontend PR.

Main changes explained:

  • Added new endpoint in TimeEntry rotuer
  • Implemented corresponding controller function

How to test:

  1. check into current branch
  2. follow instruction on frontend PR

Copy link

@Srichand-Medagani Srichand-Medagani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have reviewed this PR and the 10+ hours filter works perfectly fine. But the 'Clear' button only clears the dates set and the search bar but not the filters set. They are only cleared when the page is refreshed.

video1441872085.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants