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

Add Language Options to Settings #4112

Merged
merged 5 commits into from
Oct 27, 2023
Merged

Add Language Options to Settings #4112

merged 5 commits into from
Oct 27, 2023

Conversation

joeizang
Copy link
Collaborator

@joeizang joeizang commented Oct 4, 2023

fixes #4068

@joeizang joeizang added the WIP label Oct 4, 2023
@joeizang joeizang self-assigned this Oct 4, 2023
@netlify
Copy link

netlify bot commented Oct 4, 2023

Deploy Preview for remixproject ready!

Name Link
🔨 Latest commit 22391cd
🔍 Latest deploy log https://app.netlify.com/sites/remixproject/deploys/653bcc1f49bc12000890af88
😎 Deploy Preview https://deploy-preview-4112--remixproject.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@joeizang joeizang added ready-to-review PR ready to review and removed WIP labels Oct 12, 2023
@joeizang joeizang requested a review from ryestew October 12, 2023 10:43
@joeizang joeizang force-pushed the add-lang-options branch 2 times, most recently from 80fdd8f to 6632a25 Compare October 12, 2023 12:02
Copy link
Collaborator

@ryestew ryestew left a comment

Choose a reason for hiding this comment

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

I would like this PR to include the upcoming .json files - in Spanish and French and maybe in Chinese. These additions could be in a different PR. I'll coordinate with you about the timing of when these translations will be ready.

const _paq = window._paq = window._paq || []

const locales = [
{ code: 'en', name: 'English', localeName: 'English', messages: enJson },
{ code: 'zh', name: 'Chinese Simplified', localeName: '简体中文', messages: zhJson },
{ code: 'es', name: 'Spanish', localeName: 'Español', messages: esJson },
Copy link
Collaborator

Choose a reason for hiding this comment

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

These should be alphabetized - well at least alphabetized in English - so the first entry of this list would be chinese.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

@joeizang joeizang requested a review from ryestew October 26, 2023 09:34
@joeizang joeizang force-pushed the add-lang-options branch 2 times, most recently from bf852fc to b67341e Compare October 26, 2023 16:33
@ryestew
Copy link
Collaborator

ryestew commented Oct 27, 2023

we can add Italian to this list

@joeizang joeizang merged commit fbc6d2b into master Oct 27, 2023
5 checks passed
@joeizang joeizang deleted the add-lang-options branch October 27, 2023 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-review PR ready to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Spanish, French, & Italian to Language options in the Settings panel
2 participants