Skip to content

Conversation

@alexandre-combemorel
Copy link
Contributor

currently if files are not named with basic language 2 letters length (such as 'en', 'fr', ...) then default locales are not found.

If we have files matching the languages pattern of browser, such as (fr-FR, en-US, etc) then it was not taken into account.

This PR fix this

@changeset-bot
Copy link

changeset-bot bot commented Jul 10, 2024

🦋 Changeset detected

Latest commit: f894a02

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@scaleway/use-i18n Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link

codecov bot commented Jul 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.48%. Comparing base (a06a685) to head (f894a02).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2096   +/-   ##
=======================================
  Coverage   77.48%   77.48%           
=======================================
  Files          25       25           
  Lines         755      755           
  Branches       94       94           
=======================================
  Hits          585      585           
  Misses        162      162           
  Partials        8        8           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alexandre-combemorel alexandre-combemorel changed the title fix/add default locales from browser to be able to select them fix(i18n): add default locales from browser to be able to select them Jul 10, 2024
@philibea philibea merged commit a86bb14 into main Jul 12, 2024
@philibea philibea deleted the feat/make-default-browser-translations-available branch July 12, 2024 07:08
@github-actions github-actions bot mentioned this pull request Jul 9, 2024
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.

3 participants