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

Documentaçao do endpoint de consultar redes sociais do usuário #50

Merged
merged 5 commits into from
May 10, 2024

Conversation

joaomarcelo09
Copy link
Contributor

@joaomarcelo09 joaomarcelo09 commented May 2, 2024

Closes #17

Feature

Implementação da documentação para utilizar o endpoint de consultar rede sociais do usuário

Bugfix
  • Description
    N/A

  • Cause
    N/A

  • Solution
    N/A

Changelog N/A
Visual evidences 🖼️
Checklist
  • Issue linked
  • Build working correctly
  • Tests created
Additional info N/A

Copy link
Member

@zoldyzdk zoldyzdk left a comment

Choose a reason for hiding this comment

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

Coloca uma descriçãozinha nesse PR meu mano, nos outros prs tem um template pra tu se basear.

@joaomarcelo09
Copy link
Contributor Author

Coloca uma descriçãozinha nesse PR meu mano, nos outros prs tem um template pra tu se basear.

feito meu bom

Copy link
Contributor

@alvarogfn alvarogfn left a comment

Choose a reason for hiding this comment

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

Está tudo ok com o código do seu PR

Você só precisa ajustar a nomenclatura que utilizou:

Ao invés de usar

user-network-consult

Substitua por

user-network-find-all

Consult não é um nome comumente usado e isso pode dificultar o entendimento no futuro.

@kvnbg kvnbg self-requested a review May 6, 2024 09:40
@joaomarcelo09 joaomarcelo09 requested a review from alvarogfn May 6, 2024 17:27
@alvarogfn alvarogfn enabled auto-merge (squash) May 7, 2024 02:46
auto-merge was automatically disabled May 10, 2024 17:30

Head branch was pushed to by a user without write access

@zoldyzdk zoldyzdk merged commit ba9c98a into devhatt:main May 10, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
5 participants