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

Add Japanese translation of README #5468

Merged
merged 2 commits into from Jan 23, 2021
Merged

Add Japanese translation of README #5468

merged 2 commits into from Jan 23, 2021

Conversation

ghost
Copy link

@ghost ghost commented Jan 21, 2021

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

Translate README into Japanese

Fixes the following issue(s)

Relies on the following changes

Due diligence

@XiangRongLin XiangRongLin added the localisation / translation Everything that has to do with translations or Weblate label Jan 21, 2021
Copy link
Collaborator

@XiangRongLin XiangRongLin left a comment

Choose a reason for hiding this comment

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

Isn't the iso code for japan jp and not ja?

@ghost
Copy link
Author

ghost commented Jan 21, 2021

Yes. The country code for Japan is jp, but the language code is ja.

Copy link
Collaborator

@XiangRongLin XiangRongLin left a comment

Choose a reason for hiding this comment

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

Well then i learned something.

I don't know how to remove my "changes requested" review, so just dismiss my review

@triallax
Copy link
Contributor

@Ryota-Hasegawa could you also please add a link to README.pt_BR.md?

@ghost
Copy link
Author

ghost commented Jan 23, 2021

@mhmdanas Done.

Copy link
Contributor

@triallax triallax left a comment

Choose a reason for hiding this comment

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

LGTM

@triallax triallax requested a review from XiangRongLin January 23, 2021 07:56
@triallax
Copy link
Contributor

@XiangRongLin can you figure out how to remove your requested changes and approve the PR instead? I can't merge the PR unless you do so.

@XiangRongLin
Copy link
Collaborator

@mhmdanas done

@opusforlife2 opusforlife2 merged commit 9ada979 into TeamNewPipe:dev Jan 23, 2021
This was referenced Jan 23, 2021
@ghost
Copy link
Author

ghost commented Jan 23, 2021

Thank you for merging :)

tossj pushed a commit to tossj/NewPipe-legacy that referenced this pull request Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
localisation / translation Everything that has to do with translations or Weblate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants