diego3g / rsxp-2023

Repositório contendo o código do app do RS/XP 2023
MIT License
443 stars 85 forks source link

feat: add Sympla service #78

Closed gabe-frasz closed 1 year ago

gabe-frasz commented 1 year ago

📋 Descrição

Criei o serviço de comunicação com a API do Sympla com um único método que retorna as informações do participante de acordo com o número do ingresso. Para isso, instalei e configurei o axios e adicionei duas novas variáveis ambiente (definidas no .env.example). Por fim, incluí o serviço do Sympla no app.module.ts.

Também adicionei a tipagem da resposta da API com os campos que achei mais relevante, mas vou deixar essa parte e a possibilidade de adicionar novos métodos em aberto e sujeitos à mudança por enquanto.

Fixes #65

🛠️ Tipo da mudança

🧪 Como isso foi testado?

Realizei testes manuais.

✅ Checklist:

jorge-lba commented 1 year ago

@gabe-frasz seria legal ver com o @diego3g se não seria dar um nome diferente como em vez de service usar provider.

Falo isso pq o nest utiliza service para adicionar lógicas de negócio ( parecido com o use case da arquitetura limpa ) e acho que isso pode confundir o pessoal, por utilizar dois termos iguais para finalidades diferentes

gabe-frasz commented 1 year ago

@gabe-frasz seria legal ver com o @diego3g se não seria dar um nome diferente como em vez de service usar provider.

Falo isso pq o nest utiliza service para adicionar lógicas de negócio ( parecido com o use case da arquitetura limpa ) e acho que isso pode confundir o pessoal, por utilizar dois termos iguais para finalidades diferentes

Certo, vamos aguardar a resposta dele.

E sobre a funcionalidade em si, você acredita que haja a necessidade de adicionar um novo método que possa ser necessário no futuro?

jorge-lba commented 1 year ago

@gabe-frasz acredito que para cada item da lista que temos de back so sympla seria uma função no service( ou provider heheh )

Mas ai não necessariamente uma pessoa precise fazer todas

eletroswing commented 1 year ago

@gabe-frasz se ajudar, aqui está um pull que eu ia fazer mais cedo, por não ter visto este. Se conseguir tirar algo para ajudar.. Pull request 87.

gabe-frasz commented 1 year ago

Boa tarde, pessoal. Fiz algumas pequenas alterações de acordo com os feedbacks.

diego3g commented 1 year ago

@gabe-frasz podes resolver os conflitos? Do package-lock.json é só apagar o arquivo e rodar um npm install do zero.

gabe-frasz commented 1 year ago

Resolvidos, @diego3g! 💪

diego3g commented 1 year ago

tem um pouquinho mais @gabe-frasz