Skip to content

VC/Zoom: Add meeting registration sync#292

Open
moliholy wants to merge 25 commits intoindico:masterfrom
moliholy:feat/registrations
Open

VC/Zoom: Add meeting registration sync#292
moliholy wants to merge 25 commits intoindico:masterfrom
moliholy:feat/registrations

Conversation

@moliholy
Copy link
Copy Markdown
Contributor

This PR enables registration in zoom meetings when they get registered in indico events with an associated zoom meeting.

@moliholy moliholy force-pushed the feat/registrations branch from 16eb1ad to fb4807c Compare January 7, 2026 18:38
@moliholy moliholy marked this pull request as ready for review January 25, 2026 14:45
@moliholy
Copy link
Copy Markdown
Contributor Author

moliholy commented Jan 25, 2026

@ThiefMaster this changes the flow to enforce registration_required to always be true. I was thinking to add a per-event setting to enable the old workflow with approval_type equal to 2 (no registrations required). What do you think?

@ThiefMaster
Copy link
Copy Markdown
Member

Yes, this needs to be an event setting for sure. Default should be to NOT require registration and make this new workflow opt-in. I'm fine w/ having a plugin-level setting to configure the default.

@moliholy
Copy link
Copy Markdown
Contributor Author

@ThiefMaster added b15f807 to include a plugin-wide setting to enable/disable registrations. Disabled by default.

@moliholy
Copy link
Copy Markdown
Contributor Author

@ThiefMaster friendly ping 🙏

@moliholy moliholy force-pushed the feat/registrations branch from 1e87704 to b410151 Compare March 27, 2026 11:12
@moliholy moliholy force-pushed the feat/registrations branch from b60b305 to 2a7e397 Compare March 29, 2026 01:27
@moliholy
Copy link
Copy Markdown
Contributor Author

@ThiefMaster ready for another round!

@moliholy moliholy force-pushed the feat/registrations branch from 5d1e6a9 to fdb2039 Compare March 29, 2026 01:59
@ThiefMaster ThiefMaster changed the title feat: enable registrations on zoom when users get registered in indico VC/Zoom: Add meeting registration sync Mar 30, 2026
- indentation
- usefixtures instead of unused arg
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.

2 participants