Skip to content

Conversation

henrif75
Copy link
Collaborator

@henrif75 henrif75 commented Mar 8, 2024

Professional translations for CR2 content.
Changes are being staged in a separate branch until all Japanese content is reviewed.

#1463 #652

@henrif75 henrif75 marked this pull request as ready for review March 8, 2024 22:06
Copy link
Collaborator

@HidenoriKobayashi HidenoriKobayashi left a comment

Choose a reason for hiding this comment

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

Just comments. OWNERS have to agree.

Copy link
Collaborator

@HidenoriKobayashi HidenoriKobayashi left a comment

Choose a reason for hiding this comment

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

Just comments. OWNERS have to agree.

@henrif75
Copy link
Collaborator Author

Just comments. OWNERS have to agree.

Hi @HidenoriKobayashi thanks so much for your help!
One tip is that you can use suggest code changes that can be directly applied to the code. Check out this help article, thanks!

@HidenoriKobayashi
Copy link
Collaborator

Hi @HidenoriKobayashi thanks so much for your help! One tip is that you can use suggest code changes that can be directly applied to the code. Check out this help article, thanks!

Thanks for the tip @henrif75 ! I didn't know that you can make a suggestion that spans multiple lines using the codespace. I'll do that next time to make everyone's life easy :)

Copy link
Collaborator

@HidenoriKobayashi HidenoriKobayashi left a comment

Choose a reason for hiding this comment

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

I briefly went though the slides for Day 2 again.

Besides some suggestions (some are due to my bad comments, sorry), I found some places the original English text is shown in the browser.

I think we can merge this in the next round. Thanks!

@henrif75 henrif75 requested a review from HidenoriKobayashi May 1, 2024 19:00
@henrif75
Copy link
Collaborator Author

henrif75 commented May 1, 2024

I briefly went though the slides for Day 2 again.

Besides some suggestions (some are due to my bad comments, sorry), I found some places the original English text is shown in the browser.

I think we can merge this in the next round. Thanks!

Hi @HidenoriKobayashi I believe we took care of all comments. The problem where some parts are still in English might be solved when we refresh and msgmerge the .po file.
One more think, can you check this to solve the CLA issue?

Thanks!

@HidenoriKobayashi
Copy link
Collaborator

Hi @HidenoriKobayashi I believe we took care of all comments. The problem where some parts are still in English might be solved when we refresh and msgmerge the .po file. One more think, can you check this to solve the CLA issue?

Thanks!

Thanks for the fixes! I think this chapter is ready to merge.

Re: CLA issue, go/github shows my private account is linked, and I have my chromium email added to my private account. I had no problem contributing other google projects. Could you help figuring out what I'm missing? @henrif75

Besides, I don't seem to have the permission to approve. I thought Martin had added me to the project (#652 (comment)). Maybe I needed to do something on my side to actually join this project?

@henrif75
Copy link
Collaborator Author

henrif75 commented May 7, 2024

Hi @HidenoriKobayashi I believe we took care of all comments. The problem where some parts are still in English might be solved when we refresh and msgmerge the .po file. One more think, can you check this to solve the CLA issue?
Thanks!

Thanks for the fixes! I think this chapter is ready to merge.

Re: CLA issue, go/github shows my private account is linked, and I have my chromium email added to my private account. I had no problem contributing other google projects. Could you help figuring out what I'm missing? @henrif75

Besides, I don't seem to have the permission to approve. I thought Martin had added me to the project (#652 (comment)). Maybe I needed to do something on my side to actually join this project?

Hi @HidenoriKobayashi you should have gotten an invitation from GitHub to join the repository as a writer, when you accept it the issue with approvals should be fixed.

I'll try to figure out what's going on with the CLA and I'll circle back.

Copy link
Collaborator

@HidenoriKobayashi HidenoriKobayashi left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

henrif75 and others added 4 commits May 7, 2024 20:00
Co-authored-by: Hidenori Kobayashi <hidenorik@chromium.org>
Co-authored-by: Hidenori Kobayashi <hidenorik@chromium.org>
@henrif75 henrif75 merged commit a286c9b into google:jp-cr2 May 8, 2024
@henrif75 henrif75 deleted the jp-cr2-day3 branch May 8, 2024 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants