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

[PT-BR-brazilian portuguese] translation #244

Conversation

gabryellekrueger
Copy link

@gabryellekrueger gabryellekrueger commented Nov 1, 2021

I'm a brazilian developer interested in data science. I've translated a part of the project to Brazilian Portuguese and plan on helping more, so more brazillians can have access to the lectures :)

#136

@ghost
Copy link

ghost commented Nov 1, 2021

CLA assistant check
All CLA requirements met.

Copy link
Member

@paladique paladique left a comment

Choose a reason for hiding this comment

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

@gabryellekrueger thank you so much for your contribution! This file is in the translations directory, which is why the images and links are broken. Once those are updated this will be ready to merge.

@@ -0,0 +1,147 @@
# Trabalhando com dados: Dados não-relacionais

|![ Ilustrado por [(@sketchthedocs)](https://sketchthedocs.dev) ](../../sketchnotes/06-NoSQL.png)|
Copy link
Member

Choose a reason for hiding this comment

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

Since this is in the translations folder, the path to the images has changed so be sure to update them. This is the case for the other images in here as well.


### Tópicos

1. [Bancos de dados relacionais](05-relational-databases/README.md)
Copy link
Member

Choose a reason for hiding this comment

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

These links are broken and return a 404. If there are pt-br versions of the files, these paths can be updated to those. If not, you can update the path to the English version until those files are translated.

@jlooper
Copy link
Contributor

jlooper commented Mar 7, 2022

closing for staleness

@jlooper jlooper closed this Mar 7, 2022
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.

3 participants