Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Aplicação Django como módulo único #285

Merged
merged 2 commits into from
Jan 16, 2021
Merged

Aplicação Django como módulo único #285

merged 2 commits into from
Jan 16, 2021

Conversation

cuducos
Copy link
Contributor

@cuducos cuducos commented Jan 15, 2021

Esse PR tem como objetivo transformar o Django em módulo único, como descrito em #71.

Coisas pendentes:

  • testar manualmente as alterações (por enquanto, só confiei no pytest)
  • reverter o commit 5126594 quando tivermos uma nova versão do Model Bakery publicada
  • resolver conflitos com a main
  • resolver erro do makemigrations --check (obrigada CI, quem sou eu sem vc? 🙇 )

Alguém de disponibiliza a me ajudar com os testes manuais?

@anapaulagomes
Copy link
Contributor

Posso ajudar! :D Como tu quer fazer? Quer q eu faça uns testes ou testar junto? Pode me pingar no telegram e a gente combina um horário qq coisa.

Essa atualização possibilita transformar a aplicação web em módulo
único. Para mais detalhes:

model-bakers/model_bakery#139
@cuducos
Copy link
Contributor Author

cuducos commented Jan 15, 2021

Como tu quer fazer? Quer q eu faça uns testes ou testar junto?

Olha, eu não me importo que testem e coloquem os casos de erro aqui. Assim seguimos assincronamente. Para mim, seria melhor assim nesse momento : )

@anapaulagomes
Copy link
Contributor

Sem problemas, @cuducos! Vou olhar e posto aqui. Obrigada!

@cuducos
Copy link
Contributor Author

cuducos commented Jan 15, 2021

Me desculpem. Fiz uns push --force pq deixei passar umas coisinhas enquanto resolvia conflito com a main. Agora creio estar mais limpa a branch : )

@cuducos cuducos marked this pull request as ready for review January 15, 2021 19:03
@anapaulagomes
Copy link
Contributor

anapaulagomes commented Jan 15, 2021

Ao tentar rodar os testes localmente (make tests depois de buildar etc) rolou o seguinte erro:

docker-compose run --rm web pytest
Creating maria-quiteria_web_run ... done
Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 908, in _find_spec
AttributeError: 'ConfigurationImporter' object has no attribute 'find_spec'

During handling of the above exception, another exception occurred:
[...]
  File "/usr/local/lib/python3.8/site-packages/configurations/importer.py", line 139, in find_module
    imp.find_module(module, path))
  File "/usr/local/lib/python3.8/imp.py", line 269, in find_module
    raise RuntimeError("'path' must be None or a list, "
RuntimeError: 'path' must be None or a list, not <class '_frozen_importlib_external._NamespacePath'>
make: *** [Makefile:29: tests] Error 1

Mas fazendo um clone do projeto e checkout da branch rolou direitinho. 🤔

@anapaulagomes
Copy link
Contributor

No repo limpo, testei a nossa pequena API, o painel, admin e o management command que roda os crawlers e salva no banco e também o que atualiza os índices da FTS e está tudo certo. Nenhum erro. 🥇

@cuducos
Copy link
Contributor Author

cuducos commented Jan 16, 2021

Massa!

Sobre o commit 5126594, como preferem fazer? Mandamos para a branch principal assim mesmo (instalando do GitHub mesmo) e depois que tiver release do Model Bakery a gente reverte? Ou deixamos esse PR amarrado até sair a nova release do Model Bakery?

@anapaulagomes
Copy link
Contributor

Prefiro mergear logo pra gente ter um feedback e atualizar os outros PRs. Obrigada, @cuducos!

@anapaulagomes anapaulagomes merged commit c253b75 into DadosAbertosDeFeira:main Jan 16, 2021
@anapaulagomes
Copy link
Contributor

Deploy ✅

@anapaulagomes
Copy link
Contributor

Os crawlers estão rodando em produção sem nenhum erro. 🥇

@cuducos
Copy link
Contributor Author

cuducos commented Jan 18, 2021

Uhuu… ganhei meu dia : )

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants