Conversation
…ed template in the env
|
The first thing is that OCATOM branch says:
That means, that the branch should be synchronized with master first, i.e. master have to be merged into OCATOM: git checkout master
git pull
git checkout OCATOM
git merge masterthis will:
as a result OCATOM should be ahead of master by one yours commit. |
majkelx
left a comment
There was a problem hiding this comment.
Make changes and push to OCATOM branch and/or discuss my comments
| echo "Collecting static..." | ||
| sleep 3 | ||
| done | ||
| #while ! python manage.py collectstatic --no-input 2>&1; do |
There was a problem hiding this comment.
collectstatic, zależnie od serwera na którym uruchamiany jest projekt, może kopiować statyczne pliki tak żeby nie były serwowane przez pythona tylko bezpośrednio. (wźcej poszukaj info o django collectstatic)
Nie wiem jak chłopaki to u siebie organizują w dev, ale też nie wywalałbym im tego bez porozumienia - jak Ci to sprawiało problemy to ich zapytaj.
ewentualnie cały katalog docker/dev można skopiować na docker/oca_dev i tam hulaj dusza, ale i tak trzeba zromieć zarządanie staticami przez django generalnie (z doc) i w kontekście bhtoma (od chłopaków)
There was a problem hiding this comment.
Nie kopiujemy static, używamy ścieżki w której one są, udostępniamy je przez nginx location i alias. Co do drugiej sprawy, może faktycznie lepiej by było docker/oca_dev, bo trudno powiedzieć, w którym kierunku będą szły nasze zmiany w przyszłości - ale nie upieram się.
| django-pgviews==0.5.7 | ||
| django-tables2==2.4.1 | ||
| factory_boy==3.2.1 | ||
| fits2image==0.4.4 |
There was a problem hiding this comment.
O ile nie potrzebujesz podnosić wersji zależności, nie mieszalbym tego tutaj.
Jeśli nie ma bezpośredniego powodu przywróciłbym oryginalne requirements.txt
There was a problem hiding this comment.
nie mogłem odpalić projektu lokalnie bez podnoszenia wersji
…ed template in the env