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

Feat/merge backend frontend #5

Merged
merged 4 commits into from
Apr 12, 2022
Merged

Conversation

glaubervila
Copy link
Contributor

  • Movi o frontend pz-server -> frontend
  • Movi o backend pz-server-api -> backend
  • Criei um arquivo env_template com as variaveis de ambiente (facilita o deploy pq o nome do arquivo será sempre o mesmo).
  • Fixei a versão do postgres para 13.6 similar a nossa versão de produção que é 13.4.
  • Fixei a versão da imagem python do backend para python 3.10 a mesma que estava sendo utiliza antes.
  • Alterei a criação da imagem do backend para utilizar a versão slim.
  • A imagem do backend está usando um venv para as dependencias, isso resolve o warning do pip sobre o uso do root. a vantagem é que a imagem é mais segura já que não é mais possivel adicionar libs depois do build. a desvantagem é que qualquer nova dependencia deve ser adicionada ao requirements.txt e depois feito um build.
  • Mudei o diretório static do django para archive/django_static dessa a mudança do nome é por que o react usa um diretório static também e isso causa conflitos.
  • Mudei as urls do django rest, todas agora tem o prefixo api/. essa mudança é pq o frontend assume a url raiz /.
  • Adicionei um ngnix ao docker-compose, deve ser ligado sempre. ( deixei o pzserver em http://localhost) a porta 80 pode ser mudada no docker-compose caso já esteja em uso. internamente o ngnix está rodando na 8080.

…frontend. in this commit the backend is ready.
… the same project, a docker compose was created for development with the environment all configured.
@crisingulani crisingulani merged commit 1ca9116 into main Apr 12, 2022
@glaubervila glaubervila deleted the feat/merge_backend_frontend branch April 18, 2022 14:48
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