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

Import - Relax "is_required" check for custom fields #23576

Merged
merged 1 commit into from
May 26, 2022
Merged

Import - Relax "is_required" check for custom fields #23576

merged 1 commit into from
May 26, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

image

@civibot
Copy link

civibot bot commented May 25, 2022

(Standard links)

@civibot civibot bot added the master label May 25, 2022
@totten totten changed the title Remove required field check on import Import - Relax "is_required" check for custom fields May 25, 2022
@monishdeb
Copy link
Member

Agree with the fix, did an r-run in local, using the required Yes/No custom field. Works fine doesn't cause any regression. Merging now

@monishdeb monishdeb merged commit d6994c3 into civicrm:master May 26, 2022
@eileenmcnaughton eileenmcnaughton deleted the cust_it branch May 26, 2022 05:40
@eileenmcnaughton
Copy link
Contributor Author

yay - thanks @monishdeb - now I'm gonna remove that field from the testdata extension cos it makes the site a nightmare to use

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eileenmcnaughton @monishdeb