Skip to content
Snippets Groups Projects

Tabulka Moje soutěže

Closed Jan Prachař requested to merge honza/mr7 into devel
1 unresolved thread

Tohle mělo u organizátorů pozitivní ohlas. Hned po přihlášení vidí soutěže, které se jich týkají. K dokonalosti by to ale chtělo nějaký tuning.

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
  • První dojmy: líbí se mi přehled soutěží na hlavní stránce (viz ostatně #107 (closed)), ale dost váhám, jestli v tom přehledu chci mít také nějaké ovládací prvky. Přijde mi, že je lepší, když se akce vyvolávají z jednoho dobře definovaného místa. Ještě rozmyslím, až nebudu napůl spát.

  • Martin Mareš
    Martin Mareš @mj started a thread on the diff
38
39 contests = []
40 for role in roles:
41 q = (sess.query(db.Contest, db.UserRole.role, db.Round)
42 .select_from(db.Contest)
43 .filter_by(place_id=role.place_id)
44 .join(db.UserRole, db.UserRole.user_role_id == role.user_role_id)
45 .join(db.Round))
46 if role.year:
47 q = q.filter(db.Round.year == role.year)
48 if role.category:
49 q = q.filter(db.Round.category == role.category)
50 if role.seq:
51 q = q.filter(db.Round.seq == role.seq)
52 contests += q.options(joinedload(db.Contest.place)).all()
53
  • Tady bych se vyhnul generování dotazů v cyklu. Rolí někteří organizátoři mají docela dost. Nemělo by to být problém přepsat na subquery.

    Také pozor na to, že jednu soutěž můžeš najít přes několik různých rolí.

  • Předělal jsem to na jedno (resp. dvě query). Nějaký hrdina to může zkusit přepsat na jedno sqlalchemy query :)

  • Please register or sign in to reply
  • Martin Mareš
  • Martin Mareš
  • Celkově jsem pro, ovládací prvky vyzkouším, až pomergeujeme věci, na kterých tenhle patch závisí :)

  • Jan Prachař added 1 commit

    added 1 commit

    Compare with previous version

  • Jan Prachař added 47 commits

    added 47 commits

    • d10072e4...e286d393 - 44 commits from branch devel
    • c47a0c4b - Rights: Funkce, která vrátí důvod, proč se nezobrazilo zadání
    • ebf2817b - org_index: Tabulka Moje soutěže na úvodní stránce
    • dd95eeb9 - Tabulka Moje soutěže

    Compare with previous version

  • Jan Prachař added 1 commit

    added 1 commit

    Compare with previous version

  • Takhle například to vypadá (jenom tam nemám barvičky stavů):100: Screenshot_from_2021-04-19_23-43-29 Screenshot_from_2021-04-19_23-52-20

  • Martin Mareš mentioned in merge request !80 (merged)

    mentioned in merge request !80 (merged)

  • closed

  • Please register or sign in to reply
    Loading