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

[Erro] Sincronização com o tópico oficial #62

Merged
merged 3 commits into from
Jul 17, 2016
Merged

[Erro] Sincronização com o tópico oficial #62

merged 3 commits into from
Jul 17, 2016

Conversation

khaosdoctor
Copy link
Contributor

Estavam faltando alguns tópicos que estão no original, criei eles tanto neste readme quanto no da pagina inicial. É melhor dar uma olhada nos outros também

Pareciam estar faltando alguns tópicos nessa lista, adicionei eles aqui.
Alguns links estavam faltando no Native, adicionei eles lá e aqui
3.39 Toast
3.40 TouchID
3.41 Vibration
3 Nativo

Choose a reason for hiding this comment

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

Acho que pode tirar essa linha, a de cima "####3. Nativo" já está identificando o capítulo.

Choose a reason for hiding this comment

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

Vai ter que incluir mais uma linha aqui do 3DTouch, mas não precisa ser nesse PR... Pode ser em outro.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Eu acho que identifica melhor o capítulo, ou então a gente pode tirar os readmes por capitulo de dentro do repositório e só deixar o readme principal apontando pra eles.

@gabrielclima
Copy link

Eles realmente estão incluindo mais coisas.. De vez em quando alguém vem e atualiza os links, temos que ficar de olho mesmo.

Por mim, depois de tirar aquela primeira linha, já pode fazer o merge...

@khaosdoctor
Copy link
Contributor Author

Mas ai não vai ficar meio "perdido"? Tipo, jogado os números dos tópicos

@gabrielclima
Copy link

Desculpa, eu escrevi de uma forma meio confusa.. Quis dizer que o "####3. Nativo" já está fazendo a função de título, o "3 Nativo" pode ser retirado, saca?
image

Agora está assim
image

Seu PR está assim
image

É como se tivessem 2 títulos pro capítulo

@khaosdoctor
Copy link
Contributor Author

Nossa cara eu realmente não tinha visto isso kkk Pronto, removido!

@gabrielclima
Copy link

Haha sem problemas!

@gabrielclima gabrielclima merged commit 475df9f into IonicBrazil:master Jul 17, 2016
@gabrielclima gabrielclima self-assigned this Jul 17, 2016
@khaosdoctor khaosdoctor deleted the docSync branch July 17, 2016 23:12
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.

2 participants