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

Adiciona lógica de consumo da API de convite da Portfoliorrr #62

Merged
merged 7 commits into from
Feb 2, 2024

Conversation

oLucasAguilar
Copy link
Contributor

@oLucasAguilar oLucasAguilar commented Feb 1, 2024

Alcançamos com este PR

Resolve #57

Esse PR adiciona a lógica de consumo da API de criação de convite da Portfoliorrr.

Até então, quando um convite era criado, isso era feito apenas na ColaBora. Então a ideia é que, com essa PR, quando um convite for criado ele só será válido se for criado nas duas plataformas.

Criamos uma model nova chamada PortfoliorrrInvitation onde criamos o método responsável por fazer a requisição POST na API da Portfoliorrr.

A chamada da requisição foi inserida dentro do método de criação de um convite. Ele é criado com o status processing e só recebe o status pending quando recebemos um retorno positivo da API. Se a API não retornar success o convite é automaticamente cancelado.

Devido a essa necessidade do retorno da requisição, tivemos que adicionar em alguns testes o allow para o Faraday mockar a resposta positiva da API. Ficamos em dúvida de qual seria a melhor prática para os testes passared, então usamos o allow mesmo.

Deixamos a estrutura dos métodos de create e cancel do Invitation preparadas para a criação dos jobs.

Débitos

  • Ficou em débito o consumo da API patch da Portfoliorrr para cancelamento de convite, para que o status dos convites nas duas plataformas fiquem sincronizado.

Luckvc and others added 3 commits February 1, 2024 11:51
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Copy link
Contributor

@paulohenrique-gh paulohenrique-gh left a comment

Choose a reason for hiding this comment

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

Ficou ótimo pessoal! Solicitei apenas um ajuste no teste que ficou com nome duplicado. Sobre a dúvida no allow, @oLucasAguilar, não sei se entendi bem mas me parece estar no padrão que a gente já vinha usando. Se a intenção era usar spy, o que entendi foi que ele é usado pra monitorar as chamadas de métodos em uma classe, e o allow pra manipular o que um método retorna. Desculpa se entendi errado, qualquer coisa a gente se fala. Bom trabalho!

spec/system/invitations/user_send_invitation_spec.rb Outdated Show resolved Hide resolved
- Mostra mensagem correta para o usuário
- Resolve conflitos com a main

Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Copy link

@OtavioLins OtavioLins left a comment

Choose a reason for hiding this comment

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

Com exceção de uma questão de estrutura, o código me parece bem bom! Bom trabalho :)

app/models/portfoliorrr_invitation.rb Outdated Show resolved Hide resolved
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
@Luckvc Luckvc requested a review from OtavioLins February 2, 2024 14:23
Copy link

@OtavioLins OtavioLins left a comment

Choose a reason for hiding this comment

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

Ficou ótimo, gente. Vocês não conheciam service mas basicamente fizeram um, só que deram o nome de model :)

Luckvc and others added 2 commits February 2, 2024 11:45
Para melhor organização do código com adições futuras.

Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
@Luckvc Luckvc merged commit e826ec2 into main Feb 2, 2024
2 checks passed
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.

[Integração] Enviar convite para Portfoliorrr
4 participants