fix: make checkin button click once to prevent submitting multiple logs #2604
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ruchamahabal
pushed a commit
that referenced
this pull request
Jan 14, 2025
…gs (backport #2604) (#2605) Co-authored-by: Asmita Hase <[email protected]>
There was a problem hiding this comment.
An ideal fix would be to set loading state on button while the document is being inserted. Example here
hrms/frontend/src/views/Login.vue
Lines 29 to 35 in cf3266b
frappe-pr-bot
pushed a commit
that referenced
this pull request
Jan 14, 2025
# [15.38.0](v15.37.2...v15.38.0) (2025-01-14) ### Bug Fixes * add leaves to the correct allocation for compensatory leave request ([19304f4](19304f4)) * allow setting opening tax balance in salary assignment for old employees too (backport [#2618](#2618)) ([#2636](#2636)) ([d99f5e3](d99f5e3)) * Breadcrumb Label for Non-Logged-In Users in Job Portal (backport [#2439](#2439)) ([#2598](#2598)) ([72875c6](72875c6)) * create single leave ledger encashment entry for carry forwarding leave type (backport [#2614](#2614)) ([#2627](#2627)) ([44acce7](44acce7)) * disable 'User Cannot Search' for bulk tools (backport [#2630](#2630)) ([#2632](#2632)) ([5098f4f](5098f4f)) * effect of fnf statement cancellation on gratuity record ([c4f4999](c4f4999)) * filter out cancelled documents in FnF reference document selection ([870bbaa](870bbaa)) * make checkin button click once to prevent submitting multiple logs (backport [#2604](#2604)) ([#2605](#2605)) ([eb7c313](eb7c313)) * rectify and add more info to Leave Ledger validation message ([2cd858f](2cd858f)) * refactor code, consider fnf payment status update via journal entry ([e92caf4](e92caf4)) * refactor code, consider fnf payment status update via journal entry ([dad727a](dad727a)) * resolve html rendering issue for holidays in roster view (backport [#2595](#2595)) ([#2596](#2596)) ([4a2ca25](4a2ca25)) * set gratuity paid_amont field ([d64c255](d64c255)) * set gratuity to unpaid on FFN cancellation, edit test ([f53b861](f53b861)) * set party type & party for gratuity payable row in journal entry ([a9fa9fb](a9fa9fb)) * skip updating gratuity status on FnF cancellation ([b5e3a4d](b5e3a4d)) * update status of reference documents ([746d628](746d628)) * validate department based on company ([bc2fdbd](bc2fdbd)) * validate if company is set in department ([99065c5](99065c5)) ### Features * add company filter for department ([09d21d2](09d21d2)) ### Reverts * Revert "fix: refactor code, consider fnf payment status update via journal entry" ([0ad0848](0ad0848))
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
Checkin button submits a log for every subsequent click after the first, leading to multiple checkin logs with close enough timestamps in a throttled network.
Before
before.mov
After
after.mov