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

Invitations2024 #2486

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from
Draft

Invitations2024 #2486

wants to merge 12 commits into from

Conversation

nchiapol
Copy link
Contributor

I cherry-picked the changes from /pull/2054 and /pull/2053 into a new new branch on top of master.

  1. this pull request adds the option to "mass-invite" persons to an event to the "zu Veranstaltung hinzufügen" button on the people tab:
    screenshot_2024-03-11_000326
  2. this pull request adds a download button to the invitiation tab of events:
    screenshot_2024-03-11_000547

In my manual tests, everything still works, so I think the "merge" worked.
Additionally I tried to add specs. The basic adding of an invitiation works, but I got stuck with the tests for the two new features. (comments in the new spec file explain where I got stuck)

@TheWalkingLeek
Copy link
Member

Hier gibt es meines Erachtens nach noch zu viele Baustellen als dass das Merge-Ready wäre (Specs, Export als Background Job, ToDos im code etc.)..
Würde es eventuell Sinn machen ein Ticket für diesen Use Case zu erstellen (falls es dies noch nicht gibt), den PR dort zu verlinken und dann auf "Draft" stellen? :)

@nchiapol
Copy link
Contributor Author

nchiapol commented Mar 26, 2024

Einverstanden, dass das noch nicht bereit ist. Ob wir die Diskussion hier oder in einem Ticket führen ist für mich beides ok. Das Ticket besteht bereits: #1279. Habe dort aber keine besonderen Rechte. (d.h. ich weiss nicht, wie ich diesen Pull-Request richtig verlinken kann - oder wie ich den auf Draft umstelle.)

Betreffend Export im Background. Gemäss Gespräch mit Carlo am Hackaton ist hier vorerst auch ein direkter Export ok.

@TheWalkingLeek TheWalkingLeek marked this pull request as draft March 26, 2024 11:38
@TheWalkingLeek TheWalkingLeek linked an issue Mar 26, 2024 that may be closed by this pull request
@TheWalkingLeek
Copy link
Member

Super also, ich habe das demnach mal zum Draft konvertiert und verlinkt :)

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.

EVENTS: Mehrere Personen gleichzeitig einladen
2 participants