Skip to content
Snippets Groups Projects

Formulář na přidání účastníka do soutěže

Closed Jan Prachař requested to merge honza/add-contestant into devel
2 unresolved threads

Zněužil jsem za tím účelem třídu Import, tak nevím, jak se vám to bude líbit :)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Martin Mareš
  • Martin Mareš
  • Martin Mareš
  • Martin Mareš
  • Martin Mareš
  • Martin Mareš
  • Martin Mareš
  • 252 sess.query(db.Place)
    253 .select_from(db.Participation).join(db.Place)
    254 .filter(db.Participation.contest_id == contest_id).all()
    255 )
    256 if len(places) == 0:
    257 places = (
    258 sess.query(db.Place)
    259 .select_from(db.Contest).join(db.Place)
    260 .filter(db.Contest.contest_id == contest_id).all()
    261 )
    262
    263 self.participation_place.choices = [(p.place_id, p.name) for p in places]
    264 if site_id:
    265 self.participation_place.data = site_id
    266
    267 def validate_email(form, field):
  • Jan Prachař added 5 commits

    added 5 commits

    • 7c8bc3b1 - add participant: fix link from participant table
    • 57492d77 - do not import Required
    • 9fa5b55d - typo
    • 4d7884fc - superfluous permissions check
    • 2f089720 - selectbox i pro školy na úrovni okresu a výše

    Compare with previous version

  • Jan Prachař added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Mareš
  • Jan Prachař added 1 commit

    added 1 commit

    • dd4fbca7 - Přidání účastníka: style

    Compare with previous version

    • Tohle bych rád mergenul, ale jednak to má konflikty se současným devel a jednak je to ještě stále moc chlupaté. Především nerozumím, proč je potřeba definovat tři různé formuláře.

    • Dotáhnu to. Ale nevím, jak s tím selectboxem pro školy pro okresní soutěže. Samotnýmu mi to přišlo dost praktický, když jsem jednoho jedince přidával. Vlastní kódy se moc nechytly, i když jsem na ně výslovně upozorňoval. Tohle jsou všechny zadaný organizátorama :D

           4726 | #4726
           2096 | #2096
            384 | #382
      

      (Vlastní kód, co začíná na # ani nefunguje, ne?)

    • Nefunguje :)

      Nejsem proti selectboxu na školy, jen se mi nelíbí, aby měl formulář hromadu variant. Myslím, že by mělo stačit definovat jeden formulář, jehož části se podle potřeby zobrazí nebo schovají.

      Přemýšlím, jestli se výběr školy ze seznamu nebude hodit i jinde, třeba příští rok, až se budou účastníci (aspoň do některých kategorií) přihlašovat sami. Možná nějaký kousek JavaScriptu, který si podle vybraného kraje a okresu bude umět stáhnout seznam škol a dát z něj vybrat? Zatím jen fantazíruji, konkrétní představu nemám :)

    • Tak je to. Javascript jsem teď neřešil, ale byl by fajn

    • Please register or sign in to reply
  • Jan Prachař added 192 commits

    added 192 commits

    • dd4fbca7...1ad1e1d6 - 180 commits from branch master
    • 6d50d6d5 - Hezčí čeština: oblast->okres/kraj/škola
    • cf946688 - Zrušeno staré pole place_level_names
    • 12de13ef - Bugfix volání get_level()
    • 355a9448 - Kosmetika
    • 1b29a523 - Hezčí čeština ve formátování rolí
    • e286d393 - Přesnost bodování se vždy bere z hlavního kola skupiny
    • 39143432 - Konzistentně e-mail s pomlčkou
    • 968e9c70 - Přehlednější UI pro emaily účastníků
    • 02574dc9 - Lepší předmět pro email účastníkům
    • e1fea2a7 - Emaily účastníkům: Odkaz zpět na seznam přes drobečkovou navigaci
    • 08c1dd75 - fix: self.job is None
    • 6482b1dc - Merge branch 'honza/mri' into 'devel'

    Compare with previous version

  • Jan Prachař added 6 commits

    added 6 commits

    • 8a896b0e - Formulář na přidání účastníka do soutěže
    • 0a4de0f9 - selectbox i pro školy na úrovni okresu a výše
    • bcb309a1 - Přidání účastníka: odkaz na katalog míst
    • 1da68240 - Přidání účastníka: One form to rule them all
    • e235f83f - Přidat účastníka: oprava tlačítek
    • 8faa7f83 - import: add run_on_row()

    Compare with previous version

  • Jan Prachař changed target branch from master to devel

    changed target branch from master to devel

  • Viz alternativní implementace v MR !90 (merged).

  • closed

  • Please register or sign in to reply
    Loading