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

mozilla-japan/translation#676 #11193

Merged
merged 4 commits into from
Feb 12, 2023
Merged

mozilla-japan/translation#676 #11193

merged 4 commits into from
Feb 12, 2023

Conversation

hitoshi44
Copy link
Contributor

mozilla-japan/translation#676

Description

確認のためにPR出させていただきました。以下が変更点です。

  • ディレクトリ, ファイルの作成(md, svg)
  • タイトルの変更
  • tags を l10n: hash に変更
  • 冒頭の翻訳

@hitoshi44 hitoshi44 requested a review from a team as a code owner January 21, 2023 07:45
@hitoshi44 hitoshi44 requested review from dynamis and removed request for a team January 21, 2023 07:45
@github-actions github-actions bot added the l10n-ja Issues related to Japanese content. label Jan 21, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jan 21, 2023

Preview URLs

Flaws (52)

URL: /ja/docs/Web/API/WebRTC_API/Signaling_and_video_calling
Title: シグナリングとビデオ通話
Flaw count: 52

  • macros:
    • /ja/docs/Glossary/SDP does not exist but fell back to /en-US/docs/Glossary/SDP
    • /ja/docs/Glossary/SDP does not exist but fell back to /en-US/docs/Glossary/SDP
    • /ja/docs/Web/API/RTCPeerConnection/icecandidate_event does not exist but fell back to /en-US/docs/Web/API/RTCPeerConnection/icecandidate_event
    • /ja/docs/Web/API/RTCPeerConnection/icecandidate_event does not exist but fell back to /en-US/docs/Web/API/RTCPeerConnection/icecandidate_event
    • /ja/docs/Web/API/RTCPeerConnection/addIceCandidate does not exist but fell back to /en-US/docs/Web/API/RTCPeerConnection/addIceCandidate
    • and 46 more flaws omitted
  • broken_links:
    • Can use the English (en-US) link as a fallback
External URLs (3)

URL: /ja/docs/Web/API/WebRTC_API/Signaling_and_video_calling
Title: シグナリングとビデオ通話

(comment last updated: 2023-02-12 08:14:04)

Copy link
Collaborator

@potappo potappo left a comment

Choose a reason for hiding this comment

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

13 行目がスペースだけの行になっているかと思うので、削除してください。

日本語と英数字の間には半角スペースを入れてください。
https://github.com/mozilla-japan/translation/wiki/Editorial-Guideline

@mfuji09
Copy link
Collaborator

mfuji09 commented Feb 9, 2023

@hitoshi44 さん
potappo さんからのコメントが来てから3週間がたちました。
本件を続いて対処されますか?

@hitoshi44
Copy link
Contributor Author

@mfuji09
ありがとうございます。potappoさんからの指摘は修正済みなのですが、まとまったコミットができたら再度リクエストをだすつもりでした。
自分としては引き続き翻訳を続けるつもりです。

翻訳はまだまだ終わっておらず、空き時間に少しずつ翻訳していく予定なのですが、ちょっと時間をかけすぎでしょうか。

@mfuji09
Copy link
Collaborator

mfuji09 commented Feb 10, 2023

@hitoshi44 さん
あまり長く時間をかけると、元の英語版が変更されたり、日本語版の他の記事との整合が問題になる可能性があります。切りの良いところでいったんコミットし、残りの翻訳は別なプルリクエストで投稿するという方法もあります。

@hitoshi44
Copy link
Contributor Author

@mfuji09 さん
ありがとうございます。仰っていいただいた方法でコミットしていこうと思います。

@potappo さん
指摘箇所の修正と、(短いですが)次のセクションの翻訳を行いました。
時間が空いてしまい恐縮ですが、レビューお願いいたします。

@potappo
Copy link
Collaborator

potappo commented Feb 11, 2023

@hitoshi44 了解です!今外出中なので、夜以降、明日中にはチェックします。

Copy link
Collaborator

@potappo potappo left a comment

Choose a reason for hiding this comment

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

脱字を1箇所修正しましたが、他は問題ありませんでしたので、これでいったんマージします。

@potappo potappo merged commit 49e8d9f into mdn:main Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ja Issues related to Japanese content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants