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

I18n implementation #342

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

Conversation

NeroxTGC
Copy link
Contributor

i18n implementation

  • Browser Language detector
  • Fallback/default language: EN
  • Available languages: EN/ES
  • Added LanguageSelector to the NavBar

image
image

Translated:

  • Landing Page
  • Pricing Page
  • Account Page
  • FileUploadPage
  • DemoAppPage
  • NavBar component
  • UserMenuItems component
  • Footer component

Contributor Checklist

Make sure to do the following steps if they are applicable to your PR:

(not done)

i18n implemented:
- Languages: EN/ES
- User language detection
- Default language: EN

Translated pages:
Landing Page, Pricing, Account, FileUploadPage, DemoAppPage
@Martinsos Martinsos requested a review from vincanger December 16, 2024 20:29
@Martinsos
Copy link
Member

Cool stuff @NeroxTGC !
What do you say @vincanger ? Makes sense to me (I haven't checked the code, but as a concept). I guess the question is if it is possibly too much to put in the template, but I guess it might be fine, people have asked for it quite often? If not we can potentially move it into a guide instead hm.

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