[Feature] Criação de proposal no Cola?Bora! ao preencher formulário de solicitação de convite no Portfoliorrr #161
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…o colabora Co-authored-by: Rodrigo Gyodai <[email protected]>
Co-authored-by: Rodrigo Gyodai <[email protected]>
…posals Co-authored-by: Rodrigo Gyodai <[email protected]>
Co-authored-by: Rodrigo Gyodai <[email protected]>
Co-authored-by: Rodrigo Gyodai <[email protected]>
Co-authored-by: Rodrigo Gyodai <[email protected]>
Cria regra para tentar por 5 vezes criar um proposal no ColaBora caso o servidor deles esteja fora do ar Cria regra para repetir solicitação à API do Portfoliorrr sem limites de tentativas caso dê erro interno do servidor Cria regra para mudar o status de invitation_request para error caso a API do ColaBora retorne um erro 4XX Também foi necessário alterar 2 testes de sistema que não estavam passando por conta de um bug causado pela forma como eles foram escritos (alinhado com os responsáveis pelos testes). Co-authored-by: Rodrigo Gyodai <[email protected]>
Co-authored-by: Rodrigo Gyodai <[email protected]>
Modifica o texto informado ao cliente para instrui-lo de onde acompanhar a evolução de sua solicitação de convite Co-authored-by: Rodrigo Gyodai <[email protected]>
eliseuramos93
changed the title
eliseuramos93
requested review from
rozbr96,
akaninja,
DanSmaR,
gabdemiranda,
hreis1,
caiquedv,
gmkoeb,
Luckvc,
paulohenrique-gh and
anaresgalla
gmkoeb
approved these changes
Feb 12, 2024
paulohenrique-gh
approved these changes
Feb 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Essa PR aborda e resolve #120
Objetivos Alcançados
Com essa PR, completamos a comunicação entre Portfoliorrr/Cola?Bora! no envio de solicitação de convite através da página de projetos do Portfoliorrr. Agora, ao preencher o formulário no Portfoliorrr e salvá-lo, além de criar uma instância de
invitation_requestem nosso banco de dados também é enviada uma solicitação de criação deproposal(equivalente aoinvitation_request) na plataforma co-irmã.Toda vez que uma instância de
invitation_requesté criada (seja através do formulário, via seeds, console, etc.) é enfileirado um job emRequestInvitationsJob. Com um servidor de jobs ativo, esses jobs executam e enviam a requisição à API do Cola?Bora!, para criação doproposalno banco de dados deles.Foram implementadas todas as regras de negócio planejadas dessa ação no contexto atual, contemplando casos de sucesso completo, tratando exceções de indisponibilidade do servidor do Cola?Bora!, indisponibilidade do nosso servidor, e eventuais erros causados pelo usuário (409) e também eventuais erros de envio de dados por parte do Portfoliorrr (404).
Foram realizados testes com as duas aplicações em execução, e tivemos êxito na criação de uma instância de
proposalno Cola?Bora! através do formulário existente no Portfoliorrr.Apesar da informação de parâmetros não permitidos, o
project_idfoi recebido corretamente e a solicitação de convite foi criada com sucesso, conforme status201e print abaixo.Conforme alinhamento feito com a equipe do Cola?Bora!, já estamos enviando o parâmetro
invitation_request_idpara aplicação deles, mesmo que ela não esteja sendo absorvida no momento em que essa PR está sendo revisada/aprovada. Já consta no roadmap deles receber essa informação e, como enviar o parâmetro agora não teria nenhum impacto no funcionamento da entrega, optamos por realizar a entrega com esse parâmetro já sendo enviado.Critérios de Aceite
Ver issue #120
Regras de negócio implementadas
Status de solicitações de convite (enum)
Ver issue #120
Débitos
aborted,errorourefused. Isso precisa ser alterado em PR futura.error, será discutido se isso será priorizado antes da entrega final da primeira versão da app