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

Fixed Spanish Translation Errors in Project #1485

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

AdityyaX
Copy link

@AdityyaX AdityyaX commented Jan 3, 2024

Fixes #1447

Changes:

Spanish Translation in books section corrected
src/data/es.yml modified

Screenshots of the change:

Screen.Recording.2024-01-03.at.10.52.44.AM.mov

Request for Review: @limzykenneth @Guirdo @Andreu-G @holomorfo

Copy link
Contributor

@holomorfo holomorfo left a comment

Choose a reason for hiding this comment

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

Hi, I have added some corrections in the plural/singular matching of some words and suggested a few word ordering that I believe improves your translation. @AdityyaX
Please let me know what you think. Thanks

src/data/es.yml Outdated Show resolved Hide resolved
src/data/es.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@holomorfo holomorfo left a comment

Choose a reason for hiding this comment

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

Great, thanks

@AdityyaX
Copy link
Author

AdityyaX commented Jan 3, 2024

Thank you for your assistance, @holomorfo;
I have implemented the suggested improvements.

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.

Translation Issue
2 participants