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

feat(guest): allow to add multiple at once #74

Open
dargmuesli opened this issue Oct 11, 2024 · 2 comments · Fixed by #153
Open

feat(guest): allow to add multiple at once #74

dargmuesli opened this issue Oct 11, 2024 · 2 comments · Fixed by #153
Assignees
Labels
feature New feature or request technical Not a feature nor a bug

Comments

@dargmuesli
Copy link
Member

dargmuesli commented Oct 11, 2024

Right now there is only an automatically generated endpoint for invitation creation that creates a single invitation at once. Inviting many guests needs many calls to that endpoint which is inefficient and might even lock users out for some time due to rate limiting. There should be a new function that expects an array of contact ids to be invited and which tries to create an invitation for all of the given contact ids at once.

Frontend issue:

@dargmuesli dargmuesli added feature New feature or request good first issue Good for newcomers hacktoberfest Recommendation for work that qualifies for hacktoberfest approvals labels Oct 11, 2024
@dargmuesli dargmuesli moved this to 🔖 Ready in maevsi Oct 11, 2024
@dargmuesli
Copy link
Member Author

cc @sthelemann, to be done after internal tasks are completed.

@dargmuesli dargmuesli changed the title feat(invitation): allow to create multiple invitations at once feat(guest): allow to add multiple at once Dec 20, 2024
@dargmuesli
Copy link
Member Author

dargmuesli commented Feb 2, 2025

Make sure to check if there is native support for such api functionality in Postgrahile before implementing an own wrapper!
See: https://postgraphile.org/postgraphile/current/function-gallery#insert-multiple-records

@dargmuesli dargmuesli added technical Not a feature nor a bug and removed good first issue Good for newcomers hacktoberfest Recommendation for work that qualifies for hacktoberfest approvals labels Feb 2, 2025
@dargmuesli dargmuesli linked a pull request Feb 13, 2025 that will close this issue
@dargmuesli dargmuesli moved this from 🔖 Ready to 🏗 In progress in maevsi Feb 13, 2025
@dargmuesli dargmuesli moved this from 🏗 In progress to 👀 In review in maevsi Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request technical Not a feature nor a bug
Projects
Status: 👀 In review
Development

Successfully merging a pull request may close this issue.

2 participants