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: Nur relevante Rollen exportieren #213

Open
7 tasks done
TheWalkingLeek opened this issue Oct 4, 2022 · 0 comments · Fixed by #215
Open
7 tasks done

PEOPLE: Nur relevante Rollen exportieren #213

TheWalkingLeek opened this issue Oct 4, 2022 · 0 comments · Fixed by #215

Comments

@TheWalkingLeek
Copy link
Member

TheWalkingLeek commented Oct 4, 2022

Dieses Umsetzungs-Issue basiert auf der Diskussion in #202

Ausgangslage

Beim Export von Personen werden aktuell in der Spalte "Rollen" sämtliche Rollen einer Person aufgelistet. Einige Personen haben viele Rollen was den Export dann sehr unübersichtlich macht.

Soll

Es werden nur die Rollen aus dem aktuellen Filter/Gruppe in der Spalte Rollen aufgelistet.

image

Dieses Verhalten kann via einer Checkbox vom Benutzer gesteuert werden. Beim Wagon Der-Mitte ist diese Option standardmässig ausgewählt, sonst nicht.

Nur Excel, CSV Exporte von Personen in:

  1. Personenlisten
  2. Abos
  3. Event-Teilnehmern Nach Absprache mit tellenberger lassen wir das für dieses Ticket erstmal bleiben, da bei dem Export sowieso eine Inkonsistenz zwischen dem Export von Group::Role und Event::Role besteht

nur noch die jeweils relevanten Rollen angezeigt werden. Heisst:

  1. Nur noch die Rollen der entsprechenden Gruppe oder des entsprechenden Filter auf der Personenliste
  2. Die Rollen welche zur Abo Zugehörigkeit führen

Tech-Spec

  • Umsetzung im Core, Standardmässig ist die Option abgewählt
  • Ermöglichen das in einem Wagon die Option standardmässig ausgewählt ist
  • Preloading der Rollen verhindern Leider schlecht möglich da die Rollen sowieso spätestens bei der Ability alle geladen werden
  • Als neuer Parameter der relevanten Tabular Exporte und deren Row Klassen das Argument options = {} einführen
  • Beim Initialisieren des Exporters (& ExporterJobs) relevant_group_ids & relevant_role_types als Optionen mitgeben

ToDo

Allgemein

  • Export::Tabular::Base#initialize, Export::Tabular::Row#initialize anpassen damit eine Liste von Rollen die angezeigt werden sollen mitgegeben werden kann
  • Export::Tabular::People::PersonRow#roles anpassen und Rollen entsprechend relevant_role_types herausfiltern. Die Argumente dürfen aber weiterhin optional bleiben
  • Im SubscriptiosController die neue Option berücksichtigen und Liste der Rollen an den Export liefern falls ausgewählt
  • Im PeopleController die neue Option berücksichtigen und Liste der Rollen an den Export liefern falls ausgewählt
  • CHANGELOG-Eintrag unter "unreleased" unten hinzufügen
  • Mit angemessener Rolle "durchklicken"
  • Prüfen ob für alle Cases Specs vorhanden sind
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants