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

Allow other plugins to set a default for multi value checkboxes #3620

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jaapjansma
Copy link

@jaapjansma jaapjansma commented Sep 18, 2020

Before

If another plugin such as Caldera CiviCRM Form Processor would set a default value for a multi checkbox, the default could only be one single value. Thus allowing checking only one of the checkboxes and not multiple at the same time.

After

If another plugin such as (Caldera CiviCRM Form Processor)[https://github.com/CiviMRF/cf-civicrm-formprocessor/] would set a default value for a multi checkbox, the default could be an array and thus multiple checkboxes would be checked.

Why is this needed?

We use calder forms for an update your profile allowing a user to update their first name, last name, email address and the list of subscribed email newsletters. The current information is retrieved from CiviCRM. For the newsletter we created a checkbox field and options for each newsletter. A user could be subscribed to multiple newsletters at the same time. (You can see the form at https://staging.civicoop.org/testpagina-inschrijven-nieuwsbrie)

@jaapjansma
Copy link
Author

I am not sure why the tests are failing

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.

1 participant