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

dev/core#155 Fix optiongroup is_reserved data and use when selecting option group for custom fields #12235

Closed
wants to merge 4 commits into from

Conversation

mattwire
Copy link
Contributor

@mattwire mattwire commented May 29, 2018

Overview

Review after #12233 is merged.

As the is_reserved flag is not being shown in the UI, and is not editable from the UI it has not been used for anything in core even though it is there.

A common use-case is adding an option group and then adding a set of custom fields that use that option group (eg. for surveys) but that is currently not possible via the UI because the query that looks for available option groups is looking ONLY for option groups that are already linked to custom fields - a catch 22!

Before

Not possible to create optiongroups via the UI that can be used with custom fields.

After

Option groups that are created via the UI can be selected for use with custom fields.
customfieldoptiongroup

@mattwire mattwire changed the title Fix optiongroup is_reserved data and use when selecting option group for custom fields dev/core#155 Fix optiongroup is_reserved data and use when selecting option group for custom fields May 31, 2018
@eileenmcnaughton
Copy link
Contributor

eileenmcnaughton commented Jun 8, 2018

@mattwire this one is stale at the moment due to upgrade script changes. I'm going to close it for now - let's re-open once #12233 is merged

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