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

Add ApplicationSection and RecurringReservation Pindora GraphQL info #1610

Closed

Conversation

matti-lamppu
Copy link
Collaborator

@matti-lamppu matti-lamppu commented Feb 26, 2025

Reopened #1568 after it was mistakingly closed

🛠️ Changelog

  • Add Pindora info to the application section and recurring reservation GraphQL types
  • Add new lookup properties to recurring reservations for determining their access type based on their reservations
  • Add new Pindora fields to admin panel

🧪 Test plan

  • Automated tests

🚧 Dependencies

  • None

🎫 Tickets

@matti-lamppu matti-lamppu added the feature Adds a new feature label Feb 26, 2025
@matti-lamppu matti-lamppu self-assigned this Feb 26, 2025
@matti-lamppu matti-lamppu marked this pull request as ready for review February 26, 2025 08:47
- Don't show info from reservations in application sections before
  they are sent
- Add series and reservation ids to validity lists for identifying
  which series and reservation they correspond to
- Filter out which reservations in series/sections need access codes
  in the PindoraClient
@matti-lamppu matti-lamppu force-pushed the series-pindora-graphql branch from 6e823f0 to 928e3fb Compare February 26, 2025 09:08
@matti-lamppu matti-lamppu force-pushed the add-reservation-unit-access-type branch from 915afe0 to c3bb1f5 Compare February 26, 2025 12:43
@matti-lamppu
Copy link
Collaborator Author

Merged to #1591

@matti-lamppu matti-lamppu deleted the series-pindora-graphql branch February 26, 2025 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants