Skip to content

Conversation

@duartegalvao
Copy link
Member

@duartegalvao duartegalvao commented Jul 16, 2025

Closes #6803

This PR adds the ability to bind registrations to the email address in the invitation. This is useful for protected events when the invite is set to skip the access check, as well as managed registration forms when the invite is set to skip moderation.

image image

@github-actions github-actions bot added the alembic Contains database changes label Jul 16, 2025
@duartegalvao duartegalvao changed the title [WIP] Add option to bind registrations to invitation email Add option to bind registrations to invitation email Jul 29, 2025
@duartegalvao duartegalvao marked this pull request as ready for review July 29, 2025 10:06
Copy link
Member

@ThiefMaster ThiefMaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd s/bind/lock/

@ThiefMaster ThiefMaster force-pushed the bind-invite-email branch 2 times, most recently from 680b9ea to d0b3612 Compare August 28, 2025 11:00
description=_('If enabled, the user will be able to register even if the event '
'is access-restricted.'))
lock_email = BooleanField(_('Lock email address'), widget=SwitchWidget(),
description=_('If enabled, the email address cannot be changed during registration.'))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wondering... if modifications are allowed, then you could simply register with that email and change it afterwards.

Is this a problem? I'm leaning towards "no" because first of all the original person would get a notification and them or the event organizers would probably quickly realize that someone is doing something funny there and remove that registration.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I guess the intention was to prevent ppl mistakenly sharing that link for others to register.. It should still be fine if that person wants to change their email after registering

@ThiefMaster ThiefMaster added this to the v3.3 milestone Sep 3, 2025
@ThiefMaster ThiefMaster merged commit eff39f5 into indico:master Sep 3, 2025
10 checks passed
@ThiefMaster ThiefMaster deleted the bind-invite-email branch September 3, 2025 13:20
data-initial-values="{{ initial_values | tojson | forceescape }}"
data-management="{{ management | tojson | forceescape }}"
data-moderated="{{ moderated | tojson | forceescape }}"
data-lock-email="{{ invitation.lock_email | tojson | forceescape }}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, this broke all registration forms w/o an active invitation. luckily it showed up on the i18n-demo instance first before I put this in production (or in a release) ;)

fixed in 6efb53a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alembic Contains database changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the possibility to bind registrations to invitation email

3 participants