Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wydzielić edycję WWW jako model #79

Open
mrowqa opened this issue May 12, 2016 · 2 comments
Open

Wydzielić edycję WWW jako model #79

mrowqa opened this issue May 12, 2016 · 2 comments

Comments

@mrowqa
Copy link
Collaborator

mrowqa commented May 12, 2016

Aktualnie informacja o roku pojedynczego warsztatu jest zdublowana znajdując się w WorkshopCategory oraz WorkshopType. Ponadto informacja o uczestnikach z danego roku znajduje się w WorkshopUserProfile. Warto by było zebrać tę informację do jednego modelu Edition, do którego linkowałyby by modele WorkshopUserProfile oraz Workshop. Dodatkowo trzeba napisać migrację, która zachowa spójność danych.

@zielmicha
Copy link
Collaborator

zielmicha commented May 12, 2016

Przerost formy nad treścią IHMO (normalizacja to nie święty gral).

@sebastianjaszczur
Copy link
Owner

sebastianjaszczur commented May 12, 2016

To ułatwi filtrowanie warsztatów czy uczestników po roku, a to jest używane prawie wszędzie w aplikacji. Tzn. widzę to jako rozwiązanie w oczywisty sposób lepsze - jedynie kwestia ile pracy warto w to włożyć. Jako taki refactoring to nie jest krytyczna sprawa, ale ułatwi pisanie/zmiany kodu w przyszłości. Zobaczymy czy będziemy mieli ręce do zrobienia tego. Ja propsuję.

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

No branches or pull requests

3 participants