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

Set participant status notification to false by default #18544

Conversation

magnolia61
Copy link
Contributor

Overview

The checkbox to send a notification with some participant status changes is checked by default. For many organizations this had led to emails that were send by accident, where other

Before

The 'Send notification' is checked by default when changing a status to 'cancelled' or from 'waitlist' to 'pending waitlist'

After

The checkbox is presented but not checked by default. Sending an email should be a conscious choice.

Participant_Status_Default_notification

Technical Details

Changed the default value in templates/CRM/Event/Form/Participant.tpl

Comments

@civibot
Copy link

civibot bot commented Sep 21, 2020

(Standard links)

@civibot civibot bot added the master label Sep 21, 2020
@magnolia61
Copy link
Contributor Author

PR in the slipstream of https://lab.civicrm.org/dev/core/-/issues/1986 (on removing default checked checkbox for recurring contributions)

@pradpnayak
Copy link
Contributor

I partly agree with this change as there has been complaints from Circle clients as well but some clients prefer to have it set by default. So we ended up creating extension to include JS to set the box untick.

@eileenmcnaughton
Copy link
Contributor

thanks @magnolia61 I'll include it in dev-digest this week to see if more people support this. Also good to see @pradpnayak has already confirmed from his clients POV

@eileenmcnaughton
Copy link
Contributor

test this please

@magnolia61
Copy link
Contributor Author

I think this change makes civicrm also more consistent as all other notification boxes (eg. send contribution receipt) is also unchecked by default and it makes sense to make a deliberate choice in when to send a notification. I hope on some more support to be able to have this merged :-)

@eileenmcnaughton
Copy link
Contributor

I included this in the dev-digest. We didn't get any further response and I agree with the reasoning above, merging

@eileenmcnaughton eileenmcnaughton merged commit 6f4ad20 into civicrm:master Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants