Translate two pages and some MDX tags by smikitky · Pull Request #451 · reactjs/ja.react.dev
Navigation Menu
{{ message }}
reactjs / ja.react.dev Public
- Notifications You must be signed in to change notification settings
- Fork 186
Merged
Translate two pages and some MDX tags#451
Translate two pages and some MDX tags#451
Conversation
Copy link
Member
smikitky
commented
Dec 1, 2021
smikitky
commented
とりあえずβ版サイトを2ページ翻訳しました。先に数ページ翻訳してから reactjs/react.dev#4135 で頼めばブランチプレビューを(手作業で)セットアップしてくれるそうです。
ローカル環境では cd beta; yarn; yarn dev で正しく表示されています。
- Gotcha! は「落とし穴」とか「ハマりポイント」とか考えましたがしっくりこなかったので無難に「注意!」にしてみました。
- イタリック が環境によって表示されない問題は今後改めて修正ということで。
- prettier は off にしておかないといろいろな問題が出るようです。
- textlint も動いていませんがその対応は改めて。
smikitky
added
the
v2-translation
label
smikitky
requested review from
koba04 and
potato4d
github-actions
bot
added
the
beta
label
smikitky
removed
the
v2beta
label
koba04
self-assigned this
smikitky
mentioned this pull request
koba04 approved these changes Dec 3, 2021
Copy link
Member
koba04
left a comment
koba04
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!! 👍 👍 👍
smikitky
merged commit
6776163
into
main
smikitky
deleted the
beta-first-translation
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment