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

fix(i18n): zh-hans not working #26

Merged
merged 1 commit into from
Oct 25, 2021
Merged

fix(i18n): zh-hans not working #26

merged 1 commit into from
Oct 25, 2021

Conversation

tx0c
Copy link
Contributor

@tx0c tx0c commented Oct 24, 2021

  1. https://nft-develop.matters.news/ (zh-han traditional is working)
  2. https://nft-develop.matters.news/en/ (english is working)
  3. https://nft-develop.matters.news/zh-hans/ (not working if directly access)
  4. http://nft-develop.matters.news.s3-website-ap-southeast-1.amazonaws.com/en/ (working)
  5. http://nft-develop.matters.news.s3-website-ap-southeast-1.amazonaws.com/zh-hans/ (working)

it seems gatsby-theme-i18n has somewhere restriction of 2 letters only, only reproducing when deployed on S3 Cloudfront

@tx0c tx0c self-assigned this Oct 24, 2021
@tx0c tx0c force-pushed the fix/i18n-zh-not-working branch 2 times, most recently from c5eeaa2 to 2d31f58 Compare October 25, 2021 01:13
@tx0c tx0c requested a review from robertu7 October 25, 2021 01:20
@tx0c tx0c force-pushed the fix/i18n-zh-not-working branch from 2d31f58 to 8806d27 Compare October 25, 2021 01:23
1. https://nft-develop.matters.news/         (zh-han traditional is working)
2. https://nft-develop.matters.news/en/      (english is working)
3. https://nft-develop.matters.news/zh-hans/ (not working if directly access)

- [NFT Page] - I18N & Language Switching #23
- [NFT Page] - I18N & Language Switching 5pt feature request #9
@tx0c tx0c force-pushed the fix/i18n-zh-not-working branch from 8806d27 to 1601608 Compare October 25, 2021 01:25
Copy link
Contributor

@robertu7 robertu7 left a comment

Choose a reason for hiding this comment

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

🧷

@tx0c tx0c merged commit 1601608 into develop Oct 25, 2021
@tx0c tx0c deleted the fix/i18n-zh-not-working branch October 25, 2021 01:46
tx0c added a commit that referenced this pull request Oct 25, 2021
tx0c added a commit that referenced this pull request Oct 25, 2021
tx0c added a commit that referenced this pull request Oct 25, 2021
should be part of #26;

add ./.github/workflows/buildtest.yml to make sure `run build` should not fail
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