Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update employment details with in the single request#1442

Merged
apoorv1316 merged 1 commit intodevelopfrom
remove-mutiple-requests-to-update-employments
Aug 8, 2023
Merged

Update employment details with in the single request#1442
apoorv1316 merged 1 commit intodevelopfrom
remove-mutiple-requests-to-update-employments

Conversation

@prasanthchaduvula
Copy link
Contributor

Notion

https://www.notion.so/6afc9500b2cc42af86be55f37894fb09?v=185bd45b9dc24dfcad9c57c07d32a7c0&p=8ddde04f2605488089ff965654ee7cab&pm=s

What

  • Refactored bulk previous employments controller & service to handle updating current employment, adding new employment, updating existing employment and removing employment

Before

https://youtu.be/iq3ryeblx3E

After

https://youtu.be/LODZ7RXuiVo

@apoorv1316 apoorv1316 merged commit 321cfae into develop Aug 8, 2023
@apoorv1316 apoorv1316 deleted the remove-mutiple-requests-to-update-employments branch August 8, 2023 04:10
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
Refactored bulk previous employments controller & service to remove mutiple requests
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

Comments