chore: migrate alerts, reports, public urls and personal files to i18n - #9775
chore: migrate alerts, reports, public urls and personal files to i18n#9775AdityaHegde wants to merge 1 commit into
Conversation
|
@codex: review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: df07a686d2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| { year: "numeric", month: "short", day: "numeric" }, | ||
| )} | ||
| {m.public_url_access_expires({ | ||
| date: new Date($form.expiresAt).toLocaleDateString(undefined, { |
There was a problem hiding this comment.
When the app locale differs from the browser locale—for example, a Spanish-browser user explicitly selecting English—the undefined locale makes toLocaleDateString follow the browser instead of Paraglide's persisted selection, producing mixed-language text such as an English message with a Spanish-formatted date. Pass the current i18n locale (for example, from getLocale()) so the interpolated date matches the message language.
Useful? React with 👍 / 👎.
2 participants
As part of complete migration to i18n this handles alerts, reports, public urls and personal files components
Checklist: