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.

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

Enhanced L10N detection #111

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Enhanced L10N detection #111

wants to merge 1 commit into from

Conversation

dharmann
Copy link

  • Swithced priority between Laravel conf value and browser accepted languages
  • normalized L10N to ISO 639-1 format

- Swithced priority between Laravel conf value and browser accepted languages
- normalized L10N to ISO 639-1 format
Copy link

what-the-diff bot commented Jan 11, 2025

PR Summary

  • Improved Location Settings in getPreferredLocale function
    • The method for determining language preference was refined. Previously, if the program wasn't able to find a match for the preferred language, it would default to the language set within the app's configuration. Now it has a fail-safe, where if this too is not identifiable, it will default to English.
    • The function has been made more flexible. It can now also accept and correctly process more general inputs by just looking at the first part of the locale (e.g., English en from en-US) ensuring more effective communication across different language norms and standards.

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