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.

Sign up for GitHub

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

Jump to bottom

[locale] fr-ca: Match fr for ordinals #6110

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

[locale] fr-ca: Match fr for ordinals #6110

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bee7479
Update fr-ca.js to match fr.js
mrleblanc101 Mar 28, 2023
d164b56
Fix tests
Dec 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 6 additions & 1 deletion locale/fr-ca.js
View file
Open in desktop
Expand Up @@ -59,11 +59,16 @@
dayOfMonthOrdinalParse: /\d{1,2}(er|e)/,
ordinal: function (number, period) {
switch (period) {
// TODO: Return 'e' when day of month > 1. Move this case inside
// block for masculine words below.
// See https://github.com/moment/moment/issues/3375
case 'D':
return number + (number === 1 ? 'er' : '');

// Words with masculine grammatical gender: mois, trimestre, jour
default:
case 'M':
case 'Q':
case 'D':
case 'DDD':
case 'd':
return number + (number === 1 ? 'er' : 'e');
Expand Down