Odstrel Modelu Odevzdavatko #64

Merged

zelvuska merged 12 commits from odstrel_modelu_odevzdavatko into master 2 weeks ago

ledoian commented 3 weeks ago

Owner

Odstřelené modely odevzdávátko, prosím o otestování na testwebu a nasazení.

ledoian added 9 commits 3 weeks ago

792a8fa4dc pre

d1172b9d38 unm

49e93025c0 unm 2: změna jména

33381b4307 create

130907174d relink

d288fefecc Zrušení dočasných odkazů do `seminar.models.*`

Poznámky:
- `seminar.models.base` je v pořádku (není to dočasný soubor)
- dočasné importy v `seminar/models/*.py` jsou taky validní, protože
  odtamtud zmizí. Jde mi o to, aby náhodné věci buď používaly
  už-správnou aplikaci, nebo postaru `seminar.models` jako celek, aby
  během odstřelu nebylo potřeba všechny věci přepisovat.

Ano, tohle je potenciálně kontroverzní commit. Ale je dočasný a mně
poněkud rozbíjí workflow, když mi v náhodné okamžiky odmítne běžet
náhodná část webu na kterou nesahám jen kvůli tomu, že importuje věci
odněkud, odkud to nečekám.

Alternativní řešení: dát správné importy (s dočasnými
`seminar.models.*`) do správných `aplikace/models.py` už teď a
importovat věci rovnou z výsledných modulů. To zajišťuje jak
konzistenci, tak to, že při odstřelu se cesta změní na jednom očekávaném
místě (resp. spíš se prostě smaže) a všechno ostatní bude pokračovat ve
fungování.

10a824719b delete

07e9c8b7e4 manage

c7fe1cb386 post – odevzdávátko odstřeleno!

zelvuska added 1 commit 2 weeks ago

9ee82f72a7 SeminarModelBase.admin_url nyní umí i modely v jiných appkách…

zelvuska added 1 commit 2 weeks ago

c71195fba1 Merge branch 'master' into odstrel_modelu_odevzdavatko

zelvuska added 1 commit 2 weeks ago

a6085a64f4 Přepojení importů

zelvuska commented 2 weeks ago

Owner

Budiž. (Stejně se toho za dnešek ještě spousta změní. A data v databázi tam očividně zůstala.)

zelvuska merged commit b8f377b15d into master 2 weeks ago

zelvuska referenced this issue from a commit 2 weeks ago

Merge pull request 'Odstrel Modelu Odevzdavatko' (!64) from odstrel_modelu_odevzdavatko into master

The pull request has been merged as b8f377b15d.

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.