Skip to content

T1 + T2 Pedro Alan Rodrigo Henrique#5

Merged
DeivisFelipe merged 6 commits intoDeivisFelipe:masterfrom
PedroRamos360:pedroalanrodrigohenrique
May 5, 2024
Merged

T1 + T2 Pedro Alan Rodrigo Henrique#5
DeivisFelipe merged 6 commits intoDeivisFelipe:masterfrom
PedroRamos360:pedroalanrodrigohenrique

Conversation

@PedroRamos360
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Owner

@DeivisFelipe DeivisFelipe left a comment

Choose a reason for hiding this comment

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

O arquivo de rotas precisa estar dentro a pasta 'routers', também não precisa do middleware de CORS, ele foi coloca no projeto principal

@PedroRamos360
Copy link
Copy Markdown
Contributor Author

O arquivo de rotas precisa estar dentro a pasta 'routers', também não precisa do middleware de CORS, ele foi coloca no projeto principal

Adaptei pra ficar no padrão mas tentei rodar e deu o erro 'FastApi' object has no attribute 'default_reponse_class', achei o erro no seguinte stack overflow:
https://stackoverflow.com/questions/74097745/fastapi-object-has-no-attribute-default-response-class
Parece ser alguma coisa na implementação do include_router no main.py do projeto principal, acho que vai ser mais fácil só rodar o projeto como coloquei ali no README.md do que corrigir esse erro, tentei comparar com os outros projetos mas não achei nada de diferente que poderia causar isso (mantive o middleware do cors para caso tu vá rodar ele dentro da pasta do trabalho)

@DeivisFelipe
Copy link
Copy Markdown
Owner

Oi, consegue tentar arrumar ai de novo? mescla o t1 e o t2 em uma pull request só e tenta enviar de novo, da um git pull para pegar minhas correções

@DeivisFelipe
Copy link
Copy Markdown
Owner

estamos corrigindo os trabalhos

@PedroRamos360
Copy link
Copy Markdown
Contributor Author

ok, vou dar uma olhada

@PedroRamos360
Copy link
Copy Markdown
Contributor Author

T1 consegui fazer funcionar, vou tentar incluir o T2 aqui também daí tiro o outro pullrequest

@PedroRamos360 PedroRamos360 changed the title T1 Pedro Alan Rodrigo Henrique T1 + T2 Pedro Alan Rodrigo Henrique May 2, 2024
@PedroRamos360
Copy link
Copy Markdown
Contributor Author

Adicionei o T2 agora também, aparentemente tudo funcionando normal. OBS: se o yarn não funcionar pro front usar npm, pra mim o live-server deu problema com o yarn, em último caso no vscode tem uma extensão e live-server pra html tmb que funciona. As instruções pra rodar o front estão no README.md

@DeivisFelipe DeivisFelipe merged commit 5acc45e into DeivisFelipe:master May 5, 2024
@DeivisFelipe
Copy link
Copy Markdown
Owner

dei merge, mas vou deixar aberto a PR para manter a comunicação

@DeivisFelipe DeivisFelipe self-assigned this May 5, 2024
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.

2 participants