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

[CLOSED] Fixes translation error in Simplified Chinese #11361

Open
core-ai-bot opened this issue Aug 30, 2021 · 8 comments
Open

[CLOSED] Fixes translation error in Simplified Chinese #11361

core-ai-bot opened this issue Aug 30, 2021 · 8 comments

Comments

@core-ai-bot
Copy link
Member

Issue by mjaseem
Sunday Jun 18, 2017 at 13:22 GMT
Originally opened as adobe/brackets#13463


Fixes adobe/brackets/issues/13416


mjaseem included the following code: https://github.com/adobe/brackets/pull/13463/commits

@core-ai-bot
Copy link
Member Author

Comment by mjaseem
Sunday Jun 18, 2017 at 13:46 GMT


Didn't expect build to fail for a change in translation. Why does it happen?
This is my first PR to any project, btw.

@core-ai-bot
Copy link
Member Author

Comment by ficristo
Sunday Jun 18, 2017 at 16:18 GMT


If you click on the red x it redirects you to the continuos integration site we are using with the result of your PR. There are more info there.
In your case it first failed because of the api rate limit of GitHub (I think Brackets is using a free GitHub account which has a low rate limit), and now because of the CLA.

mjaseem has NOT submitted the contributor license agreement. See http://dev.brackets.io/brackets-contributor-license-agreement.html.

Also do you know someone who could review this ?

@core-ai-bot
Copy link
Member Author

Comment by mjaseem
Monday Jun 19, 2017 at 10:16 GMT


I assume the reviewer needs to be someone who's part of the project and understand Chinese. I don't know anyone like that.

I have now signed the CLA with my new username.

@core-ai-bot
Copy link
Member Author

Comment by ficristo
Monday Jun 19, 2017 at 19:14 GMT


You just need someone who knows Chinese.
Then someone of the Brackets team can merge.

@core-ai-bot
Copy link
Member Author

Comment by mjaseem
Tuesday Jun 20, 2017 at 13:41 GMT


@jiajun0308 @xiaoyu2016 Could you review this?

@core-ai-bot
Copy link
Member Author

Comment by ficristo
Tuesday Jun 20, 2017 at 17:53 GMT


Also@FallingHeart, could you review this ?

@core-ai-bot
Copy link
Member Author

Comment by FallingHeart
Wednesday Jun 21, 2017 at 00:56 GMT


of course@ficristo
the translation error has been fixed
but what am i supposed to do
or just say yes here

@core-ai-bot
Copy link
Member Author

Comment by ficristo
Wednesday Jun 21, 2017 at 17:41 GMT


Saying the translation is correct is enough.
For a more involved translation you could go to the "Files changed" tab and start a review / add comments.

@mjaseem@FallingHeart thank you!

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

No branches or pull requests

1 participant