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

ES #498: 2020 SEO Spanish Translation #2255

Merged
merged 14 commits into from
Jun 25, 2021
Merged

ES #498: 2020 SEO Spanish Translation #2255

merged 14 commits into from
Jun 25, 2021

Conversation

carloscastromx
Copy link
Contributor

Makes progress on #498

This is my first translated chapter, I would appreciate any comments.

I also found some typos in the source content in English and fixed them in this pull request for review.

Translated chapter and fixed some typos on the English chapter
@carloscastromx carloscastromx added good first issue Good for newcomers translation world wide web labels Jun 22, 2021
@carloscastromx carloscastromx self-assigned this Jun 22, 2021
Copy link
Member

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

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

Looks pretty good @carloscastromx ! Just a few small things to fix (which should also sort out that failing test) and then I'm happy from a technical point of view – but can't comment on the Spanish I'm afraid! 😁

@tunetheweb tunetheweb requested a review from a team June 23, 2021 16:02
carloscastromx and others added 10 commits June 23, 2021 11:18
Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
@carloscastromx
Copy link
Contributor Author

Thanks for the review! I think it's all ok @bazzadp 😄

Copy link
Member

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

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

LGTM!

@HTTPArchive/spanish-translators let me know if anyone has time to review this. Otherwise if I hear nothing in next few days then will merge away.

@tunetheweb tunetheweb merged commit daa2d17 into main Jun 25, 2021
@tunetheweb tunetheweb deleted the seo-2020-spanish branch June 25, 2021 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers translation world wide web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants