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.

Sign up for GitHub

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

Jump to bottom

Show a diff of registration data in notification emails #5269

Merged
merged 4 commits into from
Mar 4, 2022
Merged

Show a diff of registration data in notification emails #5269

merged 4 commits into from
Mar 4, 2022

Conversation

tomasr8
Copy link
Member

@tomasr8 tomasr8 commented Mar 2, 2022

Closes #5265

image

@tomasr8 @ThiefMaster
Show a diff in registration modification emails
97c3660 
Display a simple diff in notification emails which
shows the old and new value. This helps with quickly
figuring which fields have changed.
@tomasr8 @ThiefMaster
Fix tests
f27caf2
@ThiefMaster ThiefMaster enabled auto-merge (squash) March 4, 2022 18:01
@ThiefMaster ThiefMaster merged commit 3afb6fb into indico:master Mar 4, 2022
@tomasr8 tomasr8 deleted the reg-modify-email branch February 8, 2023 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ThiefMaster ThiefMaster ThiefMaster approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@tomasr8 @ThiefMaster