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

READMEの翻訳 #76

Merged
merged 7 commits into from
Jan 5, 2024
Merged

READMEの翻訳 #76

merged 7 commits into from
Jan 5, 2024

Conversation

gpsnmeajp
Copy link
Contributor

  • README自体の説明文を翻訳(ただし、十分に自身を持って翻訳できそうにない部分は手を付けていません)
  • 仕様一覧を翻訳済みのものに差し替え(未翻訳の項目が分かりにくくなるため要検討)
  • kind, message typesを翻訳

レビューよろしくお願いいたします。

@Hakkadaikon Hakkadaikon self-requested a review January 3, 2024 17:32
@Hakkadaikon
Copy link
Contributor

@gpsnmeajp
指摘記載しましたので、確認いただけると幸いです。
※インタレストリストとWalletの所は、修正お任せします

This was referenced Jan 3, 2024
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@gpsnmeajp
Copy link
Contributor Author

@Hakkadaikon
レビューいただきありがとうございます。指摘箇所を修正いたしましたので、再度確認いただけると幸いです。

@Hakkadaikon
Copy link
Contributor

修正確認しました。LGTM

README.md Outdated
- [NIP-10: Conventions for clients' use of `e` and `p` tags in text events](10.md)
- [NIP-11: Relay Information Document](11.md)
- [NIP-06: ニーモニックシードフレーズからの基本的な鍵導出](06.md)
- [NIP-07: web browsers向け`window.nostr`機能](07.md)
Copy link
Member

Choose a reason for hiding this comment

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

好みかもしれませんが、「ウェブブラウザ」の方が自然に思えます。どうでしょうか。

Copy link
Member

Choose a reason for hiding this comment

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

他の箇所にも記載があれば合わせる形で、見てて自然に感じるのは Web ブラウザ表記ですね

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Web ブラウザに直しておこうと思います。こういうのも
#77 に入れておきたいですね

README.md Outdated Show resolved Hide resolved
| `2` | 推奨リレー | |
| `3` | フォロー | [2](02.md) |
| `4` | 暗号化されたダイレクトメッセージ| [4](04.md) |
| `5` | 削除イベント | [9](09.md) |
Copy link
Contributor

Choose a reason for hiding this comment

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

どの kind もイベントではあるということを念頭に置くと イベント(の)削除 と訳したいきもちがあります。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

修正しました

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@penpenpng
Copy link
Contributor

#76 (comment)

イベントの削除 または イベント削除 のどちらかを好みで選んでいただければみたいなつもりでした。(もちろん今のままでも大丈夫です)

@ShinoharaTa ShinoharaTa merged commit 3e03116 into nostr-jp:main Jan 5, 2024
1 check passed
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.

5 participants