Skip to content
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

Use ISO 8601 dates #5107

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Use ISO 8601 dates #5107

wants to merge 5 commits into from

Conversation

gjvoosten
Copy link
Collaborator

@gjvoosten gjvoosten commented Mar 5, 2025

Add a dictionary setting that allows to use ISO 8601 for week numbering and the first day of the week.

Closes AB#1274

User changes

  • Depending on the dictionary setting, weeks start on Monday (ISO 8601) or Sunday (as before).

Superuser changes

  • None

Admin changes

  • None

System admin changes

  • application.yml or anet-dictionary.yml needs change
    Add this setting to the dictionary:
    […]
    useISO8601: true
    dateFormats:
      […]
  • db needs migration
  • documentation has changed
  • graphql schema has changed

Checklist

  • described the user behavior in PR body
  • referenced/updated all related issues
  • commits follow a repo#issue: Title title format and these 7 rules
  • commits have a clean history, otherwise PR may be squash-merged
  • added and/or updated unit tests
  • added and/or updated e2e tests
  • added and/or updated data migrations
  • updated documentation
  • resolved all build errors and warnings
  • opened debt issues for anything not resolved here

@gjvoosten gjvoosten force-pushed the AB-1274-use-iso8601-dates branch 2 times, most recently from 5c1753f to cda186d Compare March 5, 2025 10:13
@gjvoosten gjvoosten force-pushed the AB-1274-use-iso8601-dates branch from cda186d to 8610bfa Compare March 6, 2025 07:38
@gjvoosten gjvoosten marked this pull request as ready for review March 6, 2025 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant