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

fix(client): typo in APIs and Microservices Projects intro #42140

Merged
merged 3 commits into from
May 16, 2021
Merged

fix(client): typo in APIs and Microservices Projects intro #42140

merged 3 commits into from
May 16, 2021

Conversation

heysujal
Copy link
Contributor

Checklist:

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the main branch of freeCodeCamp.
  • All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

I fixed the typo in APIs and Microservices Projects introduction

fcc

Error can be found at line 315 of client/i18n/locales/english/intro.json and line 336 of client/i18n/locales/espanol/intro.json

@heysujal heysujal requested a review from a team as a code owner May 14, 2021 23:22
@gitpod-io
Copy link

gitpod-io bot commented May 14, 2021

@github-actions
Copy link
Contributor

Thanks for your pull-request.

We are no longer accepting changes to the non-English versions of files in parts of this codebase. This pull-request seems to change some of those. Please visit our contributing guidelines to learn more about translating freeCodeCamp's resources.

As always, we value all of your contributions.

Happy contributing!


Note: This message was automatically generated by a bot. If you feel this message is in error or would like help resolving it, feel free to reach us in our contributor chat.

@github-actions github-actions bot added platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. scope: i18n language translation/internationalization. Often combined with language type label labels May 14, 2021
@gikf
Copy link
Member

gikf commented May 15, 2021

Hey @heysujal

could you revert changes made to the Espanol locale? Non-English files should not be changed directly, if needed, they will be updated using the translating workflow.

@gikf gikf added the status: waiting update To be applied to PR if a maintainer/reviewer has left a feedback and follow up is needed from OP label May 15, 2021
@heysujal
Copy link
Contributor Author

heysujal commented May 15, 2021

@gikf I reverted my change by making another commit.

@gikf gikf added status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. and removed status: waiting update To be applied to PR if a maintainer/reviewer has left a feedback and follow up is needed from OP labels May 15, 2021
Copy link
Member

@gikf gikf left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@raisedadead
Copy link
Member

✨ ✨ AWESOME! LGTM. ✨ ✨

Thanks, @heysujal, for this pull request and for contributing to the code-base for the first time.

We will be looking forward to more contributions in future.

Cheers and happy contributing!

@raisedadead raisedadead changed the title Fixed typo in APIs and Microservices Projects intro fix(client): typo in APIs and Microservices Projects intro May 16, 2021
@raisedadead raisedadead merged commit 8ba3b8c into freeCodeCamp:main May 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. scope: i18n language translation/internationalization. Often combined with language type label status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants