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

Validation removed options from select/multiselect field #13757

Open
1 task done
kuzmany opened this issue May 16, 2024 · 0 comments
Open
1 task done

Validation removed options from select/multiselect field #13757

kuzmany opened this issue May 16, 2024 · 0 comments
Assignees
Labels
configuration Anything related to the Mautic configuration section custom-fields Anything related to custom fields feature A new feature for inclusion in minor or major releases

Comments

@kuzmany
Copy link
Member

kuzmany commented May 16, 2024

Mautic Version

5.0.x series

Way of installing

I downloaded a release from https://www.mautic.org/mautic-releases

PHP version

8.0

What browsers are you seeing the problem on?

No response

What happened?

Removing the options to select or multiselect can cause some problems. The most severe and frequently reported issue is that customers can't save their emails because those values are already in use in DC. To fix this, we should add validation.

Future steps include:

  • Adding validation for segment filters.
  • Considering validation for report filters.
  • Thinking about validation for contact field values.
  • Exploring support for modifying options.

Inspiration: #6566

How can we reproduce this issue?

Step 1: Create custom select field with option f and s
Step 2: Create email with dynamic content with content with condition for select field equal s
Step 3: Go to custom fields, and remove option s
Step 4: Then go back to email and try re-save. Nothing happen

Relevant log output

No response

Code of Conduct

  • I confirm that I have read and agree to follow this project's Code of Conduct




Care about this issue? Want to get it resolved sooner? If you are a member of Mautic, you can add some funds to the Bounties Project so that the person who completes this task can claim those funds once it is merged by a member of the core team! Read the docs here.

@kuzmany kuzmany added bug Issues or PR's relating to bugs needs-triage For new issues/PRs that need to be triaged labels May 16, 2024
@npracht npracht added feature A new feature for inclusion in minor or major releases configuration Anything related to the Mautic configuration section custom-fields Anything related to custom fields and removed needs-triage For new issues/PRs that need to be triaged bug Issues or PR's relating to bugs labels May 16, 2024
@npracht npracht self-assigned this May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
configuration Anything related to the Mautic configuration section custom-fields Anything related to custom fields feature A new feature for inclusion in minor or major releases
Projects
None yet
Development

No branches or pull requests

2 participants