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

Adicionando página de documentação /docs #171

Merged
merged 16 commits into from
Dec 11, 2020
Merged

Conversation

murilohns
Copy link
Collaborator

Esse PR é o mesmo que o #101, só precisei abrir outro por que acabei destruindo o outro sem querer :(

O autor dele é o @CarlosZiegler e aproveitei pra adicionar ele como contribuidor no README :)

Desculpa @CarlosZiegler pela confusão

@murilohns murilohns added the documentation Improvements or additions to documentation label Dec 10, 2020
@vercel
Copy link

vercel bot commented Dec 10, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/brasilapi/brasilapi/c6cdnezg9
✅ Preview: https://brasilapi-git-feature-add-swagger-docs.brasilapi.vercel.app

Copy link
Collaborator

@isaiasvallejos isaiasvallejos left a comment

Choose a reason for hiding this comment

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

Primeiramente @CarlosZiegler, parabéns pelo PR. Segundamente @murilohns 😂 (isaiasaisaisai)

Deixei alguns comentários de ajustes pontuais mas acho que seria legal já adicionar a documentação da rota de banks (referente ao PR #85) que foi para a master. Acho que todas as informações necessárias já estão no README.

pages/docs/doc.json Show resolved Hide resolved
pages/docs/doc.json Outdated Show resolved Hide resolved
@CarlosZiegler
Copy link
Contributor

Obrigado. Eu gostaria de saber se apenas o proprietário do PR vai como contribuidor ou outras pessoas envolvidas diretamente no PR podem ser incluidas, pergunto pois o PR foi finalizado com a ajuda do @otaciliolacerda e @RodriAndreotti . No mais muito feliz de ter ajudado.

added CarlosZiegler, otaciliolacerda and RodriAndreotti as contributors
Copy link
Member

@lucianopf lucianopf left a comment

Choose a reason for hiding this comment

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

Muito obrigado por ter atualizado o PR @murilohns e @CarlosZiegler !! ❤️

Deixei só uma pequena observação pra daí seguir com o deploy 🙏 😬

pages/docs/doc.json Outdated Show resolved Hide resolved
@allangrds allangrds mentioned this pull request Dec 11, 2020
@lucianopf lucianopf merged commit 1f4ee29 into master Dec 11, 2020
@lucianopf lucianopf deleted the feature/add-swagger-docs branch December 11, 2020 13:33
@lucianopf
Copy link
Member

MUUUITO obrigado @CarlosZiegler @otaciliolacerda @RodriAndreotti @murilohns e @isaiasvallejos ❤️ 😍 🚀

@RodriAndreotti
Copy link
Collaborator

MUUUITO obrigado @CarlosZiegler @otaciliolacerda @RodriAndreotti @murilohns e @isaiasvallejos ❤️ 😍 🚀

Opa... somos nozes!
Ainda quero pegar algo de código para mexer quando sobrar tempo (se sobrar algo, está todo mundo implementando algo...kkk)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants