Odstřel modelu TreeNode #67

Merged
zelvuska merged 5 commits from odstrel_modelu_treenode into master 1 week ago
ledoian commented 2 weeks ago
Owner

Hurá!! :-P

Ještě to chce nějaké finishing touches:

  • migraci, která obklopí všechny ostatní migrace (nevím jak, ale ten „pečlivě vymyšlený“ návrh migračních závislostí umožňuje nechat post-migrace na naprosto náhodný časový okamžik a to je blbě…
  • Překopat pořadí aplikací v mamweb/admin.py (a k tomu možná dopsat APLIKACE/apps.py pro ty věci, které nemají)
  • vymyslet, kdeže má bydlet nástřel frontendu ve Vue

Ještě nemám zkontrolováno, že se DB TreeNodů zachová, což za chvíli snad napravím :-)

Hurá!! :-P Ještě to chce nějaké finishing touches: - [x] migraci, která obklopí všechny ostatní migrace (nevím jak, ale ten „pečlivě vymyšlený“ návrh migračních závislostí umožňuje nechat post-migrace na naprosto náhodný časový okamžik a to je blbě… - [ ] Překopat pořadí aplikací v `mamweb/admin.py` (a k tomu možná dopsat `APLIKACE/apps.py` pro ty věci, které nemají) - [ ] vymyslet, kdeže má bydlet nástřel frontendu ve Vue Ještě nemám zkontrolováno, že se DB TreeNodů zachová, což za chvíli snad napravím :-)
ledoian added 2 commits 2 weeks ago
Poster
Owner

Jo, a ještě by se hodilo přeuspořádat zbytek Semináře, aby ty dva objekty nebyly ve dvou náhodných souborech – navrhuji je buď oba dát přímo do __init__.py, nebo někam do various/models/

(A beztak mě ještě napadne několik dalších drobností, co je potřeba udělat…)

Jo, a ještě by se hodilo přeuspořádat zbytek Semináře, aby ty dva objekty nebyly ve dvou náhodných souborech – navrhuji je buď oba dát přímo do `__init__.py`, nebo někam do `various/models/` (A beztak mě ještě napadne několik dalších drobností, co je potřeba udělat…)
ledoian added 1 commit 2 weeks ago
Poster
Owner

Ještě nemám zkontrolováno, že se DB TreeNodů zachová, což za chvíli snad napravím :-)

Vypadá OK (dumpy se liší podle všeho jen v pořadí sloupečků, byť všech 49229 Řešení a Hodnocení zkontrolovaných nemám :-) Ale TreeNody se zachovaly podle všeho správně.

> Ještě nemám zkontrolováno, že se DB TreeNodů zachová, což za chvíli snad napravím :-) Vypadá OK (dumpy se liší podle všeho jen v pořadí sloupečků, byť všech 49229 Řešení a Hodnocení zkontrolovaných nemám :-) Ale TreeNody se zachovaly podle všeho správně.
Poster
Owner

všech 49229 Řešení a Hodnocení zkontrolovaných nemám :-)

Mám, jsou skutečně stejné :-D (trocha sedové a awk magie :-D)

> všech 49229 Řešení a Hodnocení zkontrolovaných nemám :-) Mám, jsou skutečně stejné :-D (trocha `sed`ové a `awk` magie :-D)
ledoian added 1 commit 2 weeks ago
ledoian added 1 commit 2 weeks ago
Owner

Na testwebu padá:

django.db.migrations.exceptions.NodeNotFoundError: Migration seminar.0144_post_odstrel_vseho dependencies reference nonexistent parent node ('authtoken', '0004_alter_tokenproxy_options')

Očividně jsou na testu starší verze knihoven. (Asi závislost na verzi Pythonu, protože pip install --upgrade nefunguje… Reinstall envu pomohl (což vlastně mohlo aktualizovat verzi Pythonu, co?).)

Na testwebu padá: ``` django.db.migrations.exceptions.NodeNotFoundError: Migration seminar.0144_post_odstrel_vseho dependencies reference nonexistent parent node ('authtoken', '0004_alter_tokenproxy_options') ``` Očividně jsou na testu starší verze knihoven. (~~Asi závislost na verzi Pythonu, protože pip install --upgrade nefunguje…~~ Reinstall envu pomohl (což vlastně mohlo aktualizovat verzi Pythonu, co?).)
Owner

Takže nezapomenout při nasazení přeinstalovat env, jinak OK :)

Takže nezapomenout při nasazení přeinstalovat env, jinak OK :)
zelvuska merged commit b606f03191 into master 1 week ago
zelvuska deleted branch odstrel_modelu_treenode 1 week ago
Poster
Owner

Očividně jsou na testu starší verze knihoven. (Asi závislost na verzi Pythonu, protože pip install --upgrade nefunguje… Reinstall envu pomohl (což vlastně mohlo aktualizovat verzi Pythonu, co?).)

Myslím, že pip má nějakou opšnu na ignorování již nainstalovaných verzí – pip install --upgrade aktualizuje tuším jen to, co mu je předhozeno (tj. v našem případě requirements.txt), ale pokud nějaká knihovna má kompatibilní-leč-ne-nejnovější verzi, tak se tuším by default použije stávající a neaktualizuje se…

> Očividně jsou na testu starší verze knihoven. (~~Asi závislost na verzi Pythonu, protože pip install --upgrade nefunguje…~~ Reinstall envu pomohl (což vlastně mohlo aktualizovat verzi Pythonu, co?).) Myslím, že `pip` má nějakou opšnu na ignorování již nainstalovaných verzí – `pip install --upgrade` aktualizuje tuším jen to, co mu je předhozeno (tj. v našem případě `requirements.txt`), ale pokud nějaká knihovna má kompatibilní-leč-ne-nejnovější verzi, tak se tuším by default použije stávající a neaktualizuje se…
Poster
Owner

Hmm, ale zrovna authtoken je součástí django-rest-framework, který v requirements je. Tak nevím…

Hmm, ale zrovna `authtoken` je součástí `django-rest-framework`, který v requirements je. Tak nevím…
The pull request has been merged as b606f03191.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.