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

introducing variables that change the content depending on the select… #191

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

CHMALU
Copy link
Collaborator

@CHMALU CHMALU commented Aug 28, 2024

…ed template in the env

@majkelx
Copy link

majkelx commented Aug 28, 2024

The first thing is that OCATOM branch says:

This branch is 1 commit ahead of, 15 commits behind master.

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 master

this will:

  • update your copy of master branch to the latest version,
  • switch back to OCATOM as an active branch,
  • merge recent changes from master to your branch

as a result OCATOM should be ahead of master by one yours commit.

Copy link

@majkelx majkelx left a comment

Choose a reason for hiding this comment

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

Make changes and push to OCATOM branch and/or discuss my comments

bhtom2/templates/bhtom_common/index.html Show resolved Hide resolved
echo "Collecting static..."
sleep 3
done
#while ! python manage.py collectstatic --no-input 2>&1; do
Copy link

Choose a reason for hiding this comment

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

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)

Copy link
Collaborator

Choose a reason for hiding this comment

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

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ę.

@@ -14,10 +14,10 @@ djangorestframework>=3.12
django-pgviews==0.5.7
django-tables2==2.4.1
factory_boy==3.2.1
fits2image==0.4.4
Copy link

Choose a reason for hiding this comment

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

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

nie mogłem odpalić projektu lokalnie bez podnoszenia wersji

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

Successfully merging this pull request may close these issues.

3 participants