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

feat: add i18n support #27

Merged
merged 3 commits into from
Nov 27, 2023
Merged

feat: add i18n support #27

merged 3 commits into from
Nov 27, 2023

Conversation

nozomuikuta
Copy link
Contributor

@nozomuikuta nozomuikuta commented Nov 26, 2023

Related to #13

I added fundamental code for i18n support.
Once this PR is merged, we can start Japanese translation.

I already translated /src/ja/docs/index.md for demonstration.

I will add some inline comments.

@nozomuikuta nozomuikuta self-assigned this Nov 26, 2023
Copy link
Collaborator

@ubugeeei ubugeeei left a comment

Choose a reason for hiding this comment

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

@nozomuikuta
I've left a few comments!
Whether to address them in this PR or to consider them separately, I'd like to leave that up to you.
I'm currently thinking of making some larger changes to tsconfig, prettier config, etc., so it might be helpful for me if this gets merged first.

.vitepress/config.mts Show resolved Hide resolved
src/ja/index.md Outdated Show resolved Hide resolved
src/ja/index.md Show resolved Hide resolved
src/ja/index.md Outdated Show resolved Hide resolved
src/ja/index.md Outdated Show resolved Hide resolved
@ubugeeei ubugeeei mentioned this pull request Nov 26, 2023
@nozomuikuta nozomuikuta requested a review from ubugeeei November 27, 2023 02:21
Copy link
Collaborator

@ubugeeei ubugeeei left a comment

Choose a reason for hiding this comment

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

awesome! 👍🏻

@nozomuikuta nozomuikuta merged commit d325ce1 into main Nov 27, 2023
1 check passed
@nozomuikuta nozomuikuta deleted the feat/i18n branch November 27, 2023 02:56
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