Skip to content

fix: use UTC datetime when accumulating attendances#6

Merged
rogeriolino merged 2 commits into
v2.3from
feat/unidade-timezone
Mar 19, 2026
Merged

fix: use UTC datetime when accumulating attendances#6
rogeriolino merged 2 commits into
v2.3from
feat/unidade-timezone

Conversation

@rogeriolino
Copy link
Copy Markdown
Member

@rogeriolino rogeriolino commented Mar 17, 2026

Issue: novosga/novosga#454

Related to novosga/novosga#492

Summary

  • Replace new DateTime() (PHP default timezone) with new DateTimeImmutable('now', new DateTimeZone('UTC')) in the accumulate attendances action
  • Ensures the cutoff datetime passed to the storage layer is always UTC, consistent with how datetimes are stored in the database

Test plan

  • Trigger attendance accumulation and verify historical records are moved correctly regardless of server timezone

🤖 Generated with Claude Code

rogeriolino and others added 2 commits March 17, 2026 12:06
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@rogeriolino rogeriolino force-pushed the feat/unidade-timezone branch from 764757d to 091c162 Compare March 19, 2026 13:31
@rogeriolino rogeriolino merged commit 366f925 into v2.3 Mar 19, 2026
1 check passed
@rogeriolino rogeriolino deleted the feat/unidade-timezone branch March 19, 2026 13:40
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.

1 participant