Skip to content

Conversation

@itsbalamurali
Copy link

@itsbalamurali itsbalamurali commented May 28, 2025

What does this PR do?

Adds locale data (RelativeDateTime and Ordinals) for 40 languages as requested. This includes:
or, pa, mr, si, jv, ms, sw, ta, te, kn, ml, vi, yo, ur, gu, ro, uz, am, ha, ig, sn, kk, el, tl, he, rw, no, fi, da, lt, sl, hr, ca, sr, bg, ka, et, eu, gl.

  • RelativeDateTime implementations for these languages.
  • Unit tests have been added for the successfully integrated languages.

Important Note:
The following 6 languages are defined in the locale data but currently will not load successfully due to not being present in the intl package's default supported locale list:
jv (Javanese), yo (Yoruba), ha (Hausa), ig (Igbo), sn (Shona), rw (Kinyarwanda).
Support for these locales will require them to be added to intl or a modification to Jiffy's verifyLocale logic to handle fallbacks for date symbols more gracefully.

Issue Reference

Fixes #

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Maintenance change (non-breaking change such as upgrading a dependency, refactoring, or making a lint fix)
  • Documentation update
  • Breaking change (a fix or feature that would cause existing functionality to change)

Screenshots

If applicable, add screenshots to help explain your problem.

Checklist

  • Wrote additional tests, if needed
  • All tests have passed, you can find the scripts from the ./bin folder
  • I have updated the documentation accordingly, if needed.

Additional Information

Adds locale data (RelativeDateTime and Ordinals) for 40 languages
as requested. This includes:
or, pa, mr, si, jv, ms, sw, ta, te, kn, ml, vi, yo, ur, gu, ro,
uz, am, ha, ig, sn, kk, el, tl, he, rw, no, fi, da, lt, sl, hr,
ca, sr, bg, ka, et, eu, gl.

- RelativeDateTime implementations for these languages currently use
  English placeholder strings and will require translation.
- Ordinal data uses common defaults (empty strings or '.') and may
  need specific adjustments for some languages.
- The 'no' (Norwegian) locale has been configured to use existing
  Norwegian Bokmål ('nb') translations and settings.
- Unit tests have been added for the successfully integrated languages.

Important Note:
The following 6 languages are defined in the locale data but currently
will not load successfully due to not being present in the `intl`
package's default supported locale list:
jv (Javanese), yo (Yoruba), ha (Hausa), ig (Igbo), sn (Shona),
rw (Kinyarwanda).
Support for these locales will require them to be added to `intl`
or a modification to Jiffy's `verifyLocale` logic to handle
fallbacks for date symbols more gracefully.
@itsbalamurali itsbalamurali requested a review from jama5262 as a code owner May 28, 2025 05:48
@@ -1,5 +1,6 @@
import '../utils/verify_locale.dart';
import '../locale/relative_date_time.dart';
import '../locale/new_locale_relative_time.dart';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @itsbalamurali. Thanks for your work, first of all.

This line looks excessive. Could you, please, remove it?

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.

2 participants