Skip to content

Publish timesheet updates via socket IO#670

Merged
niraj2477 merged 12 commits intodevelopfrom
feat/socketio
Jul 3, 2025
Merged

Publish timesheet updates via socket IO#670
niraj2477 merged 12 commits intodevelopfrom
feat/socketio

Conversation

@niraj2477
Copy link
Copy Markdown
Collaborator

@niraj2477 niraj2477 commented Jul 2, 2025

Description

This pr removes manually updating timesheet data via API call, if there is any changes in timesheet, instead it now uses socketIO to publish event on which PMS app listens and update the UI with latest data.

Relevant Technical Choices

Testing Instructions

Additional Information:

Screenshot/Screencast

NA

Checklist

  • I have carefully reviewed the code before submitting it for review.
  • This code is adequately covered by unit tests to validate its functionality.
  • I have conducted thorough testing to ensure it functions as intended.
  • A member of the QA team has reviewed and tested this PR (To be checked by QA or code reviewer)

Copy link
Copy Markdown
Collaborator

@zeel-codder zeel-codder left a comment

Choose a reason for hiding this comment

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

Can we add an option to block the employee row from being clicked or edited when their approval is in progress?

@niraj2477
Copy link
Copy Markdown
Collaborator Author

Can we add an option to block the employee row from being clicked or edited when their approval is in progress?

Can look into it in next iteration

@niraj2477 niraj2477 merged commit 806a043 into develop Jul 3, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants