Skip to content

Spanish Translation Final Tasks! #158

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
6 of 10 tasks
tesseralis opened this issue Feb 16, 2019 · 30 comments
Closed
6 of 10 tasks

Spanish Translation Final Tasks! #158

tesseralis opened this issue Feb 16, 2019 · 30 comments
Assignees

Comments

@tesseralis
Copy link
Member

tesseralis commented Feb 16, 2019

¡Felicidades!

Y'all are amazing! 🎊It's been two weeks and you're translated the entire React documentation into Spanish! Give yourselves a big round of applause 👏

Now, there are couple more errands to do before we can finally deploy:

Again, amazing work, and I can't wait to introduce es.reactjs.org to the world!

@alejandronanez
Copy link
Member

It’s happening!

@alejandronanez
Copy link
Member

Thanks to @loconluis for #159!

@alejandronanez
Copy link
Member

I'll review the whole site during the weekend (life happens)! 😸

@carburo
Copy link
Member

carburo commented Feb 16, 2019

The site is deployed on Netlify so it's easier for everyone to take a look and review it.

@devrchancay
Copy link
Contributor

It looks great on Netlify but they need to review #155

@Darking360
Copy link
Collaborator

Greetings y'all 🚀 awesome! I'll take a look maybe tomorrow or Monday and see all 💪

@tesseralis
Copy link
Member Author

Would you all be okay with the current netlify deployment becoming es.reactjs.org? If so, we can get @ericnakagawa or @bvaughn to point es.reactjs.org to it.

@tesseralis
Copy link
Member Author

tesseralis commented Feb 17, 2019

I looked over it and had a few thoughts:

The headings in the home page are styled strangely?
screen shot 2019-02-16 at 6 17 43 pm
screen shot 2019-02-16 at 6 17 49 pm

We should at least translate the nav bar:

screen shot 2019-02-16 at 6 19 52 pm

And the page title:

screen shot 2019-02-16 at 6 17 34 pm

@carburo
Copy link
Member

carburo commented Feb 17, 2019

@tesseralis There is a pending PR (#160) that should translate the navbar. I should have it reviewed and merged later tonight.

@tesseralis
Copy link
Member Author

@carburo great!

Elementos DOM/DOM Elements is inconsistent: https://es-react-js.netlify.com/docs/dom-elements.html

@tesseralis
Copy link
Member Author

tesseralis commented Feb 17, 2019

PRs for:

@tesseralis
Copy link
Member Author

This is less important but we can also search for links to en.wikipedia.org or developer.mozilla.org and seeing if there are any English links that have a better Spanish version.

@tesseralis
Copy link
Member Author

One more thing: the text in the Search bar is cut off
screen shot 2019-02-17 at 1 02 57 pm

Maybe it's best to just shorten it to "Buscar"?

@alejandronanez
Copy link
Member

alejandronanez commented Feb 18, 2019

Hey @tesseralis @carburo @Darking360 @dmoralesm!

I just did a quick run through the whole website and everything seems to be ok 🎉 .

I found some minor things that could be improved and opened the respective PRs #169 #170 #171 #172

@Darking360
Copy link
Collaborator

PR #170 merged! 🚀

I'll take a look into core 💪

@Darking360
Copy link
Collaborator

Hey guys 💪 I was looking into the translation so far, I opened this PR in order to add my feedback and some things that I observed so far, I'm on React.Component right now, but I wanted you to take a look into what's there so far to not double fix the same thing.

I've seen some accents left, un-translated parts, code in spanish, some links to english docs and internal links not working for bad naming, so I've been fixing those, I hope to finish by tomorrow at noon 🤔

CC: @alejandronanez @carburo @dmoralesm @tesseralis

@lex111
Copy link
Member

lex111 commented Feb 18, 2019

I would also like to join in the congratulations to the Spanish translation team. ¡Un gran trabajo, todos! 👏

@tesseralis excuse me for writing here, but since these are final tasks, I’m wondering why there’sn’t a task to set up the search, as I understand it, need to change the Algolia API key?

@tesseralis
Copy link
Member Author

@lex111 you're totally right! We uh... should fix that.

@lex111
Copy link
Member

lex111 commented Feb 19, 2019

@tesseralis it would be nice to have instructions on how to do this 👍

@tesseralis
Copy link
Member Author

@lex111 I... have absolutely no idea.

Maybe @gaearon @ericnakagawa @bvaughn could help?

@tesseralis
Copy link
Member Author

Seems like the site uses Algolia docsearch, which is invitation only. So we can probably do a few things:

  • Have one of the maintainers apply for it pointing to es.reactjs.org. I have no idea if they’re quick at responding to requests, and it doesn’t seem like a good long term strategy for our 30+ languages
  • disable the search for now (there should be a var in the search component) so people don’t see confusing English results
  • think about switching search providers? Is that something possible? There are a few alternatives on the gatsby site.

@tesseralis
Copy link
Member Author

Eh, I’m sure it’ll be fine with Algolia. The jp.vuejs.org has it working. I’ll just have to add it to the “starting out” guide

@escorponox
Copy link
Contributor

I saw some minor issues in hooks-effect in the last review. I'll send a PR asap.

@tesseralis
Copy link
Member Author

tesseralis commented Feb 19, 2019

Guess what: https://es.reactjs.org

@bvaughn
Copy link
Contributor

bvaughn commented Feb 19, 2019

I'm trying to figure out how to force redirect HTTP -> HTTPS, but for now at least the master branch of this repo is being built by Netlify and I've setup the subdomain to point to the result.

@tesseralis
Copy link
Member Author

IIRC you have to wait until the dns propagates and then they’ll either do it automatically or there’s a big green button you can push.

@tesseralis
Copy link
Member Author

Just applied for Algolia docsearch. Should get a response in a few days. Until then, would we want to disable the search bar?

@bvaughn
Copy link
Contributor

bvaughn commented Feb 19, 2019

Yeah, thanks Nat! I was hoping that was the case based on this Stack Overflow post. Just waiting to see.

Until then, would we want to disable the search bar?

This might be good, so long as there's an issue to re-enable later.

@sergiodxa
Copy link

I saw the docs are already deployed, but it's not using HTTPS, Netlify should give a SSL cert for free with just a click in the admin panel.

@bvaughn
Copy link
Contributor

bvaughn commented Feb 20, 2019

I saw the docs are already deployed, but it's not using HTTPS

HTTPS is also available. It's just not forcefully redirecting. Supposedly this is something we can make Netlify do, I'm just not sure how yet.

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

10 participants