-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
fix: IN time not captured in Attendance through Employee Checkin #31029
Merged
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
Unrelated test failure. Merging |
mergify bot
pushed a commit
that referenced
this pull request
May 16, 2022
) (cherry picked from commit 1b7ce96) # Conflicts: # erpnext/hr/doctype/employee_checkin/test_employee_checkin.py
frappe-pr-bot
pushed a commit
that referenced
this pull request
May 17, 2022
# [13.30.0](v13.29.2...v13.30.0) (2022-05-17) ### Bug Fixes * `set_missing_values` in SE and re-use the same on all SE mappings ([fe52c1f](fe52c1f)) * Add validation for SEZ and Export invoices without payment of taxes ([cb8453d](cb8453d)) * allow to use formatting for the field to_discuss in opportunity ([e126d4e](e126d4e)) * Block 0 Qty via Update Items to be consistent with form validation ([5647875](5647875)) * Calculate totals even though pricing rule is not applied on mapped doc ([678a01d](678a01d)) * **charts:** Pass fieldtype for chart data in selling reports ([917e7c3](917e7c3)) * conflicts ([87fd933](87fd933)) * conflicts ([fb62bbf](fb62bbf)) * disable pricing rules for internal transfers (backport [#31034](#31034)) ([#31036](#31036)) ([d5eb9fb](d5eb9fb)) * discount ledger entry in case of multicurrency invoice ([#31047](#31047)) ([c3417e4](c3417e4)) * dont fail repost for recoverable errors (backport [#30979](#30979)) ([#31023](#31023)) ([a019cb6](a019cb6)) * **Employee Advance:** Return/Deduction from Salary button visibility (backport [#31011](#31011)) ([#31012](#31012)) ([5b1d85e](5b1d85e)) * Failing accounting dimension patch ([b14a7b8](b14a7b8)) * german translations for Employee ([b9bda04](b9bda04)) * gl entry validation for miniscule loan penalty ([e958ef2](e958ef2)) * hide template items from sales/purchase order ([8b99f43](8b99f43)) * IN time not captured in Attendance through Employee Checkin (backport [#31029](#31029)) ([#31031](#31031)) ([477bbcc](477bbcc)) * Item rate reset on changing posting date ([#30990](#30990)) ([8ef649f](8ef649f)) * Just add one rate in GST HSN Code ([ed76687](ed76687)) * Merge Conflicts ([3abf264](3abf264)) * Multiple fixes in GSTR-1 report ([f2cbb70](f2cbb70)) * **patch:** avoid checking for return field if it doesnt exits (backport [#30995](#30995)) ([#30997](#30997)) ([a94b5c0](a94b5c0)) * per_billed for return DN (backport [#30868](#30868)) ([#30971](#30971)) ([97ea1f5](97ea1f5)) * precision loss when transferring (backport [#30834](#30834)) ([#31032](#31032)) ([fc80a50](fc80a50)) * precision of total penalty paid ([ad21853](ad21853)) * precision of total penalty paid ([5c45737](5c45737)) * prevent bypassing forced valuation rate (backport [#30987](#30987)) ([#31020](#31020)) ([706c19d](706c19d)) * pro rata calculation for monthly depreciation ([#30989](#30989)) ([408d952](408d952)) * remove item attribute limit from variant selector (backport [#31026](#31026)) ([#31028](#31028)) ([1f016e9](1f016e9)) * Set actual qty and basic rate in SE on warehouse triggers (`get_warehouse_details`) ([30b0aee](30b0aee)) * stock analytics report shows incorrect data there's no stock movement in a period (backport [#30945](#30945)) ([#30980](#30980)) ([295ffb3](295ffb3)) * translation for status filter ([e5f8231](e5f8231)) * **translations:** Update ru translations ([#30992](#30992)) ([f797005](f797005)) * TypeError in add_indicator_for_multicompany (backport [#31042](#31042)) ([#31048](#31048)) ([e24bb1d](e24bb1d)) * unlink Attendance from Employee Checkins on cancellation (backport [#31045](#31045)) ([#31049](#31049)) ([e03fe97](e03fe97)) * UOM in HSN-wise summary of outward supply ([cd7d5cd](cd7d5cd)) * user can select disabled accounts in taxes table ([047c879](047c879)) * validate disabled accounts before posting ledger entries ([515e49b](515e49b)) * validate on hold purchase invoices in payment entry ([9fbd170](9fbd170)) ### Features * add Employee Status filter in leave balance reports ([716b525](716b525)) * add Link to Opportunity ([#30614](#30614)) ([bc23bc7](bc23bc7)) * request_for_quotation ([db4e264](db4e264)) * request_for_quotation - refactor ([b6a3e69](b6a3e69)) * select multiple values for accounting dimension (backport [#31015](#31015)) ([#31041](#31041)) ([9c21eb5](9c21eb5))
hrwX
pushed a commit
to hrwX/erpnext
that referenced
this pull request
May 20, 2022
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.
Closes #31003
Steps to replicate:
Set up Shift Type with the following options:
Create one Employee Checkin with Log Type IN and one with OUT.
Mark Attendance from Shift Type
IN Time is not set in the attendance record created
This only happens when there is just a single check-in and check-out. Usually works fine when there are multiple logs.
Fix:
Set
in_time
whenin_log
is found for the first time and there are only 2 logs.