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

PEOPLE: complete_finance Permission für IT-Support/Leitung vs. performance #219

Open
mtnstar opened this issue May 3, 2023 · 1 comment

Comments

@mtnstar
Copy link
Contributor

mtnstar commented May 3, 2023

complete_finance Permission für IT-Support/Leitung auf Bundesebene führt zu n+1 Queries beim Auflisten von Personen.
Verursacht wird das ganze durch das überarbeitete DropDown 'Rechnung erstellen'. In diesem Dropdown werden sämtliche Layer/Gruppen mit Rechnungseinstellungen aufgelistet. Die Liste ist bei Die Mitte Prod entsprechend lang:

image

Requests dauern dadurch sehr lange. (+30s) Ausserdem ist es auch extrem unpraktisch eine so lange Liste im Dropdown zu haben. (tausende von Pixel lang)

Als Quickfix haben wir die :complete_finance Permission entfernt: 79e8561

Help-Ticket: https://help.puzzle.ch/#ticket/zoom/6179

Tech-Spec

/de/groups/22/people?returning=true
eager loading detected
Group::Kanton => [:invoice_config]
Add to your query: .includes([:invoice_config])
ll stack
/usr/src/app/hitobito/app/helpers/dropdown/invoice_new.rb:99:in `block in finance_groups_items'
/usr/src/app/hitobito/app/helpers/dropdown/invoice_new.rb:98:in `map'
/usr/src/app/hitobito/app/helpers/dropdown/invoice_new.rb:98:in `finance_groups_items'
/usr/src/app/hitobito/app/helpers/dropdown/invoice_new.rb:89:in `init_items'
/usr/src/app/hitobito/app/helpers/dropdown/invoice_new.rb:20:in `initialize'

ToDo

@mtnstar mtnstar changed the title PEOPLE: complete_finance Permission für IT-Support/Leitung fü PEOPLE: complete_finance Permission für IT-Support/Leitung vs. performance May 3, 2023
@mtnstar
Copy link
Contributor Author

mtnstar commented May 4, 2023

n+1 query fix implementiert und in master gemerged: hitobito/hitobito#2100

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

1 participant