Visitar URL original
fix(lang): correct day and month spelling in eo, sq and ms by bahriy-dev · Pull Request #18596 · quasarframework/quasar · GitHub
Skip to content

fix(lang): correct day and month spelling in eo, sq and ms - #18596

Open
bahriy-dev wants to merge 1 commit into
quasarframework:devfrom
bahriy-dev:fix/lang-day-month-spelling
Open

bahriy-dev wants to merge 1 commit into
quasarframework:devfrom
bahriy-dev:fix/lang-day-month-spelling

Conversation

@bahriy-dev

@bahriy-dev bahriy-dev commented Nov 13, 2026 •

Copy link
Copy Markdown

What changes and why

Three misspelled names in the date strings of language packs:

  • eo: Thursday is Ĵaŭdo (short Ĵaŭ), not Jaŭdo.
  • sq: April is Prill, not Pril. The short form Pri was already correct.
  • ms: Friday is Jumaat, not Jummat.

All three match CLDR (Intl.DateTimeFormat with the same locale).

Breaking change?

No.

How was it validated?

Compared every language pack's date.days and date.months with Intl.DateTimeFormat output. These three were clear spelling errors; the other differences were style or regional variants and are left alone. Imported the three changed packs to confirm they load and return the new strings.

Docs

None needed.

Summary by CodeRabbit

  • Bug Fixes
    • Corrected weekday spellings in Esperanto and Bahasa Melayu date translations.
    • Corrected the Albanian spelling of December.

@coderabbitai

coderabbitai Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

Corrects weekday spellings in Esperanto and Bahasa Melayu locale data and changes an Albanian month entry from Pril to Prill.

Changes: Locale spelling corrections

Layer / File(s) Summary
Correct locale entries
ui/lang/eo.js, ui/lang/ms.js, ui/lang/sq.js
The Esperanto Thursday names change to Ĵaŭdo and Ĵaŭ. The Bahasa Melayu Friday name changes to Jumaat. The Albanian month entry changes from Pril to Prill.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Suggested labels: area/ui

Suggested reviewers: rstoenescu

Merge Risk: Minimal · up to 94f8b

This PR corrects spelling of a few day and month names in the Esperanto, Albanian and Malay language packs. It is a small data-only change with no identified merge risk.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title follows the conventional commit format with type fix and scope lang. It names the three language packs (eo, sq, ms) and the day and month spelling correction, which matches the objectives an…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Undocumented Breaking Change Passed The check is inapplicable. It covers breaking changes to public APIs: component props, events, slots, methods, CLI commands and flags, and configuration keys. The diff changes only three string litera…
Full details: Title check

Explanation

The title follows the conventional commit format with type fix and scope lang. It names the three language packs (eo, sq, ms) and the day and month spelling correction, which matches the objectives and the changed strings.

Full details: Undocumented Breaking Change

Explanation

The check is inapplicable. It covers breaking changes to public APIs: component props, events, slots, methods, CLI commands and flags, and configuration keys. The diff changes only three string literals in locale packs: ui/lang/eo.js (Jaŭdo→Ĵaŭdo, Jaŭ→Ĵaŭ), ui/lang/ms.js (Jummat→Jumaat), and ui/lang/sq.js (Pril→Prill). No component API, CLI surface, or configuration key changes. The PR description also states "Breaking change? No." Locale string values are not in the check's listed categories, so no labeling or callout requirement applies. The corrected spellings may still affect users who match on the old strings, but that is not a listed public API surface under this check.

Finishing Touches

Generate unit tests (beta)

  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 8424c2b and 94f8ba8.

Files selected for processing: 3

  • ui/lang/eo.js
  • ui/lang/ms.js
  • ui/lang/sq.js

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

This branch has not been deployed

No deployments
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