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
Explain how to use the buildCommand for internationalization #2344
Conversation
docs/Internationalization.md
Outdated
@@ -16,5 +16,11 @@ or | |||
|
|||
LOCALE=fr yarn dev | |||
|
|||
To host a localized version of Pinafore using Vercel, in the [vercel.json file](https://github.com/nolanlawson/pinafore/blob/master/vercel.json), replace the buildCommand value accordingly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than say "replace the buildCommand value accordingly," I think it would be more helpful to refer directly to the line of code:
Otherwise it's not necessarily clear what needs to be done.
docs/Internationalization.md
Outdated
There is also an experimental `LOCALE_DIRECTION` environment variable for the direction (LTR versus RTL) which is | ||
exposed to the source code while building. | ||
|
||
To host a localized version of Pinafore using Vercel, in the [vercel.json file](https://github.com/nolanlawson/pinafore/blob/master/vercel.json), replace the buildCommand key accordingly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems redundant.
Hi @nolanlawson , I've addressed your comments. Please let me know if something is still needed. |
LGTM, I am just waiting for the build to be unbroken so I can merge PRs. #2349 |
Thanks for merging and for your suggestion. I didn"t know how to provide a link to a line in a file.Enviado desde mi iPhoneEl 9 ene 2023, a las 5:02, Nolan Lawson ***@***.***> escribió:
Merged #2344 into master.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
We discussed this on PR #2329