Skip to content

Updating public api docs #588

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.

Sign up for GitHub

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

Jump to bottom
Merged
JezzDiego merged 7 commits into from
Apr 23, 2025
Merged

Updating public api docs #588

JezzDiego merged 7 commits into from
Apr 23, 2025

Conversation

@JezzDiego
Copy link
Contributor

@JezzDiego JezzDiego commented Apr 9, 2025 •
edited
Loading

  • Atualizando a documentação de todos os endpoints da public api
  • 5 endpoints utilizam o mesmo handler para a requisição, mas envolvem rotas diferentes. Criei uma função para cada endpoint para que seja possível adicionar as documentações necessárias.

@JezzDiego JezzDiego requested review from joellensilva and rdurl0 April 9, 2025 22:50
rdurl0
rdurl0 approved these changes Apr 10, 2025
View reviewed changes
Copy link

@rdurl0 rdurl0 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Excelente, Jessé!
Deixei algumas sugestões. Veja se faz sentido adicionár.

joellensilva
joellensilva approved these changes Apr 11, 2025
View reviewed changes
Copy link
Contributor

@joellensilva joellensilva left a comment

Choose a reason for hiding this comment

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

Muito bom!

@JezzDiego JezzDiego requested review from rdurl0 and talitalobo April 12, 2025 14:10
@JezzDiego
Copy link
Contributor Author

JezzDiego commented Apr 15, 2025

Chequem principalmente os comentários nos arquivos uiapi/handler.go e papi/handler.go

@JezzDiego JezzDiego requested a review from joellensilva April 17, 2025 17:18
@talitalobo
Copy link

talitalobo commented Apr 22, 2025

Você chegou a ver os comentários do pessoal, @JezzDiego ?
Achei que ficou bem detalhado!

@JezzDiego
Copy link
Contributor Author

JezzDiego commented Apr 22, 2025

Você chegou a ver os comentários do pessoal, @JezzDiego ? Achei que ficou bem detalhado!

Sim, resolvi todos esse que Raul pontuou

@JezzDiego JezzDiego merged commit ab4b16a into main Apr 23, 2025
2 checks passed
@JezzDiego JezzDiego deleted the docs-update branch April 23, 2025 14:22
@JezzDiego JezzDiego self-assigned this Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@talitalobo talitalobo talitalobo approved these changes

@rdurl0 rdurl0 Awaiting requested review from rdurl0

@joellensilva joellensilva Awaiting requested review from joellensilva

Assignees

@JezzDiego JezzDiego

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@JezzDiego @talitalobo @rdurl0 @joellensilva