Resolve "PersonalEvents and PersonalTodos get wrong time" #3717

Merged
hansegucker merged 2 commits from 1469-personalevents-and-personaltodos-get-wrong-time into master 2026-01-11 18:25:30 +01:00
Member

Information and screenshots

No further information have been provided.

Checklist

  • tox -e reformat,lint used
  • Changelog updated/No update necessary
  • Documentation (incl. Screenshots) up-to-date
  • Copyright in apps.py and README.rst up-to-date
  • (Unit) tests for new feature/regression tests for bug fixes (only if useful)
  • Adhere to URL scheme
  • Tested with admin account
  • Tested with normal account (based on permissions)

Backend checklist

  • Use permissions for all GraphQL mutations
  • Use query optimizer for all GraphQL queries
  • Don't use inefficient DB queries

Frontend checklist (mandatory with ~"part::frontend")

Closes #1469

## Information and screenshots No further information have been provided. ## Checklist - [x] `tox -e reformat,lint` used - [x] Changelog updated/No update necessary - [ ] Documentation (incl. Screenshots) up-to-date - [ ] Copyright in `apps.py` and `README.rst` up-to-date - [ ] (Unit) tests for new feature/regression tests for bug fixes (only if useful) - [ ] Adhere to [URL scheme](https://edugit.org/AlekSIS/official/AlekSIS/-/issues/86) - [ ] Tested with admin account - [ ] Tested with normal account (based on permissions) ## Backend checklist - [ ] Use permissions for all GraphQL mutations - [ ] Use query optimizer for all GraphQL queries - [ ] Don't use inefficient DB queries ## Frontend checklist (mandatory with ~"part::frontend") - [ ] Screenshots are attached (mobile and desktop) - [ ] Tested on mobile, tablet, and desktop sizes - [ ] Adhered to [frontend guidelines](https://edugit.org/AlekSIS/official/AlekSIS/-/blob/master/project/frontend_guidelines.rst?ref_type=heads) - [ ] Checked with [Firefox Accessibility Checker](https://firefox-source-docs.mozilla.org/devtools-user/accessibility_inspector/) Closes #1469
Author
Member

assigned to @yuha

assigned to @yuha
Author
Member

added 1 commit

  • e8bfa7eb - WIP: Drop local timezone conversion in calendar event mutations

Compare with previous version

added 1 commit <ul><li>e8bfa7eb - WIP: Drop local timezone conversion in calendar event mutations</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2072/diffs?diff_id=81082&start_sha=48511ff7f95d5131f9a6ccc4271b00a1bf7006cb)
Author
Member

added 1 commit

  • 38867087 - Drop local timezone conversion in calendar event mutations

Compare with previous version

added 1 commit <ul><li>38867087 - Drop local timezone conversion in calendar event mutations</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2072/diffs?diff_id=81087&start_sha=e8bfa7eb98be46575c9e2d830fae62bdd15f8e94)
Author
Member

marked the checklist item tox -e reformat,lint used as completed

marked the checklist item **`tox -e reformat,lint` used** as completed
Author
Member

marked this merge request as ready

marked this merge request as **ready**
Author
Member

marked the checklist item Changelog updated/No update necessary as completed

marked the checklist item **Changelog updated/No update necessary** as completed
Author
Member

I'm not quite sure if this is the correct way to go - but at least I've thought about and tested this extensively, and it seems to work with different client/server timezones - the client sends datetimes with its local UTC offset, the server converts them to UTC and saves them, and also delivers them this way with the client handling offsets when displaying

I'm not quite sure if this is the correct way to go - but at least I've thought about and tested this extensively, and it seems to work with different client/server timezones - the client sends datetimes with its local UTC offset, the server converts them to UTC and saves them, and also delivers them this way with the client handling offsets when displaying
Author
Member

assigned to @hansegucker and unassigned @yuha

assigned to @hansegucker and unassigned @yuha
Owner

@yuha Maybe you want to fix this?

@yuha Maybe you want to fix this?
Author
Member

resolved all threads

resolved all threads
Author
Member

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2072/diffs?diff_id=81198&start_sha=38867087aab1a71c39316e8b120df79e41155287#9bc6f6dbf7d5b3a92ff25fc3649c26a513ef5830_355_344)
Author
Member

added 1 commit

  • 5ee91aae - Drop local timezone conversion in calendar event mutations

Compare with previous version

added 1 commit <ul><li>5ee91aae - Drop local timezone conversion in calendar event mutations</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2072/diffs?diff_id=81198&start_sha=38867087aab1a71c39316e8b120df79e41155287)
Owner

added 48 commits

  • 5ee91aae...df78616a - 47 commits from branch master
  • 5a7b75e7 - Merge branch 'master' into '1469-personalevents-and-personaltodos-get-wrong-time'

Compare with previous version

added 48 commits <ul><li>5ee91aae...df78616a - 47 commits from branch <code>master</code></li><li>5a7b75e7 - Merge branch &#39;master&#39; into &#39;1469-personalevents-and-personaltodos-get-wrong-time&#39;</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/2072/diffs?diff_id=81208&start_sha=5ee91aaed7b30c5134e0a6863a5ddaf473688532)
Owner

mentioned in commit d692ce979a

mentioned in commit d692ce979a450dc7b124788a4180ca117051ff05
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aleksis/AlekSIS-Core!3717
No description provided.