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

Added reactjs spanish translation + some fixes in the english version #4220

Merged
merged 5 commits into from
Aug 14, 2022

Conversation

bandinopla
Copy link
Contributor

Translated the reactjs quiz to spanish and during that process i found and fixes some issues in the english version.
Also added myself in the creadits on the main readme file.

In regards to the fixes on the english version i found some repeated questions so opted to change the question a bit to make it unique. Some errors in the code snippeds too and lacking the language tag on them.

Copy link
Owner

@Ebazhanov Ebazhanov left a comment

Choose a reason for hiding this comment

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

Thanks for initiative to be responsible, feel free to add for more topics 😉
regarding translation i am a bit sceptical because manual translation can be different what linkedin will provide. Just FYI not sure that we have to do it with translator so the people will have hard time to match a search question.

@Ebazhanov Ebazhanov merged commit f835c01 into Ebazhanov:main Aug 14, 2022
@bandinopla
Copy link
Contributor Author

...regarding translation i am a bit sceptical because manual translation can be different what linkedin will provide. Just FYI not sure that we have to do it with translator so the people will have hard time to match a search question.

in some cases like this one i felt the question itself was wrong. It asks "when the Hello component will be displayed" but in the code it is not being rendered, just being assigned to a variable, and there's not even any code suggesting it will be indeed displayed... i mean one could asume, but the code doesn't show that. The correction i made helps the user focus on the original intended test of the question: to see if the reader understands the conditional (ternary) operator.

@Ebazhanov
Copy link
Owner

that's can happens, we are all human. As a simple solution I would prefer to add new question with correct task. And keep old version at the same time to make it searchable/match with original linkedin.

@Ebazhanov
Copy link
Owner

@all-contributors please add @bandinopla for design, code, maintenance, content

@allcontributors
Copy link
Contributor

@Ebazhanov

I've put up a pull request to add @bandinopla! 🎉

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.

None yet

2 participants