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

Require lowercase translations #150

Closed
bluzi opened this issue Oct 26, 2017 · 4 comments
Closed

Require lowercase translations #150

bluzi opened this issue Oct 26, 2017 · 4 comments

Comments

@bluzi
Copy link
Owner

bluzi commented Oct 26, 2017

  • Change all translations to be lowercase
  • Add a test that will prevent uppercase letters from appearing in a translation
@RodrigoBastos
Copy link

Hi @bluzi, I'd like to work on this.

@shashankKeshava
Copy link
Contributor

Hi @bluzi can you share more details regarding running the app after cloning and parameters for running the script. I'm finding it difficult to debug.

@bluzi
Copy link
Owner Author

bluzi commented Oct 28, 2017

Hey @shashankKeshava,

Sorry for the late response, I somehow missed your comment.

After cloning the app, you can modify collection/, and test your changes by executing npm test.
You can also launch the API by executing npm start and navigating to http://localhost:3000, there you can query names and translations.

Does that answer your question?

@bluzi
Copy link
Owner Author

bluzi commented Oct 28, 2017

BTW, this issue has solved by @rodweb, so I'm closing it, but feel free to keep asking questions in another issue, and I'll answer them gladly.

@bluzi bluzi closed this as completed Oct 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants