Skip to content

[stable33] fix(settings): display birthday correctly regardless of browsers timezone#61714

Open
backportbot[bot] wants to merge 2 commits into
stable33from
backport/61586/stable33
Open

[stable33] fix(settings): display birthday correctly regardless of browsers timezone#61714
backportbot[bot] wants to merge 2 commits into
stable33from
backport/61586/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented Jul 2, 2026

Copy link
Copy Markdown

Backport of #61586

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot Bot requested review from odzhychko, skjnldsv and susnux July 2, 2026 07:46
@backportbot backportbot Bot added bug 3. to review Waiting for reviews labels Jul 2, 2026
@backportbot backportbot Bot added this to the Nextcloud 33.0.7 milestone Jul 2, 2026
@odzhychko odzhychko force-pushed the backport/61586/stable33 branch from b18780f to 2c94283 Compare July 3, 2026 07:57
@odzhychko odzhychko force-pushed the backport/61586/stable33 branch from 2c94283 to 2df9a5a Compare July 3, 2026 08:28
odzhychko added 2 commits July 6, 2026 03:20
`savePrimaryAccountProperty` does only allow  `string | boolean`.
Incidentally, relying on `@nextcloud/axios` default serialization, `Date`s where serialized using `toISOString`.
See https://github.com/axios/axios/blob/a7d7a714498452e611943d9d8e5c2ed8b00aa6b9/lib/helpers/toFormData.js#L134

This change corrects the usage of `savePrimaryAccountProperty` and makes serialization explicit.

Signed-off-by: Oleksandr Dzhychko <hey@oleks.dev>
…zone

This also avoid shifting dates after user input.

Resolves #49828

Signed-off-by: Oleksandr Dzhychko <hey@oleks.dev>
@odzhychko odzhychko force-pushed the backport/61586/stable33 branch from 2df9a5a to 31fcb53 Compare July 6, 2026 03:24
@odzhychko

Copy link
Copy Markdown
Contributor

Ready for review. I'm fairly sure, the failing Cypress tests are not related to the changes in this PR.

@odzhychko odzhychko marked this pull request as ready for review July 6, 2026 03:55
@odzhychko odzhychko requested a review from a team as a code owner July 6, 2026 03:55
@odzhychko odzhychko requested review from sorbaugh and removed request for a team July 6, 2026 03:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant