Bug #151
open[HR][Verify] B-20: Attendance check-in times stored in two conventions (UTC vs IST-as-UTC)
0%
Description
Manual punches store true UTC; approved regularizations store IST clock time labelled as UTC. ESS shows manual punches 5.5h early, the manager list shows regularized ones 5.5h late. Reproduce on the test box: compare a manual punch and a regularized punch for the same employee in both views. Part of #140.
SV Updated by Sravan V S about 2 months ago
- Assignee set to Sravan V S
S- Updated by SK - 12 days ago
- Status changed from New to Resolved
- Assignee changed from SK - to Roshni Harish.
- Resolved By set to SK -
Already fixed - confirmed live in current dev. This is the exact bug #140 fixed earlier today (attendance time-base convention). Verified directly in the merged code: PunchRegularizationService now converts a regularization's requested time via HrIstDay.ToUtc(reg.Date, reg.RequestedIn/Out) instead of the old DateTime.SpecifyKind(..., Utc) relabel-without-converting bug - the code even carries a comment documenting exactly the bug this ticket describes. #140 also shipped a one-off data-repair migration for already-approved regularizations written under the old broken convention (flagged there for sign-off against real data before deploying).
Marking resolved on the code side. The ticket's own repro ("compare a manual punch and a regularized punch... in both views") is a live/deployed-box check - worth a QA pass on the test box once deployed, assigning to Roshni for that.
RH Updated by Roshni Harish. 3 days ago
- Assignee changed from Roshni Harish. to Sravan V S