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

Proof-of-concept for i18n support #30

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

Conversation

patcon
Copy link
Contributor

@patcon patcon commented May 24, 2018

Related: hackfoldr/hackfoldr-2.0#62

Curious if this looks like a workable solution.

@patcon
Copy link
Contributor Author

patcon commented May 24, 2018

screenshot of english translations

@irvin
Copy link
Contributor

irvin commented Jul 2, 2018

very cool, @ETBlue do you think i's good to land this?

whitelist: ['en', 'zh'],
debug: true,
backend: {
loadPath: 'https://gist.githubusercontent.com/patcon/1f68d3efd30360048bce81b3a7655c5b/raw/{{lng}}.json',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this should def point somewhere else, and prob to a raw file on github repo, or a json file hosted as part of app

Copy link
Contributor

Choose a reason for hiding this comment

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

yes I think it's better to also include these two json files in the PR

@patcon
Copy link
Contributor Author

patcon commented Jul 8, 2018

bump

would love to see this or something like it on the live site so that we can start using this more seamlessly within my english-speaking community :)

whitelist: ['en', 'zh'],
debug: true,
backend: {
loadPath: 'https://gist.githubusercontent.com/patcon/1f68d3efd30360048bce81b3a7655c5b/raw/{{lng}}.json',
Copy link
Contributor

Choose a reason for hiding this comment

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

yes I think it's better to also include these two json files in the PR

@irvin
Copy link
Contributor

irvin commented Jul 9, 2018

Also, please add some paragraphs in README.md to explain the i18n and how future contribution need to do

@patcon
Copy link
Contributor Author

patcon commented Jul 9, 2018

Thanks so much @irvin! and is this the right repo to get this into live website, or should I bring it into another repo too?

@patcon
Copy link
Contributor Author

patcon commented Sep 20, 2018

bump to above. Is this the right repo where this will get deployed to beta.hackfoldr.org? Just curious, as that is my primary motivation :)

// i18next
script(src="//unpkg.com/i18next/i18next.js")
script(src="//unpkg.com/i18next-xhr-backend/i18nextXHRBackend.js")
script(src="//unpkg.com/i18next-browser-languagedetector/i18nextBrowserLanguageDetector.js")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

is it ok to pull these from a CDN?

@caasi
Copy link
Member

caasi commented Oct 6, 2018

@irvin

The current change works on my machine. I can't reproduce your deploying problem.

Which script do you run when you deploying the app?

@irvin
Copy link
Contributor

irvin commented Oct 6, 2018

@irvin

The current change works on my machine. I can't reproduce your deploying problem.

Which script do you run when you deploying the app?

You don’t have this babel error?
#31

Also, when I run ./deploy.sh it will have an jade error prevent me from building release html to gh-pages branch, which I should open a new issue and discuss there

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