Skip to content

Conversation

@KRobertK13
Copy link
Collaborator

No description provided.

@SepsiLaszlo
Copy link
Collaborator

SepsiLaszlo commented Apr 15, 2022

TODOS

  • Keressünk szebb megoldást a Team model default scope-jára.
    • A következő sql megoldja a problémát, ezt kéne rails-es ActiveRecord függvényekből összerakni "SELECT \"teams\".* FROM \"teams\" INNER JOIN \"memberships\" ON \"memberships\".\"team_id\" = \"teams\".\"id\" GROUP BY \"teams\".\"id\" HAVING (count(memberships.id)>0)"
    • Itt tudsz jobban utána nézni: Active Record Query Interface
  • Az event model-hez nincs felvéve a teams association, ezt vegyük fel. https://guides.rubyonrails.org/association_basics.html#the-has-many-through-association
  • Az EventsController#show action-jéhez tartozó viewban szükségünk az entry-re és az ehez tartozó team-ekre is. Nem a legszebb megoldás a Team modellen keresztül kiszedni a csapatokat és után visszakeresni az entry-ket. Jobb lenne, ha az Entry modellen is lenne default_scope, ami csak az aktív csapatokat mutatja.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants