We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
In the WP Mailchimp settings, it's still possible to "include" a merge field that is has it's "visible" property unchecked in a Mailchimp account.
Note: The non visible merge fields will not display in the block editor or the front end
Replace the checkbox with "Not visible" text to inform the user that they must change the option in their account.
If we wanted to go the extra mile we could see if we could generate a link to that users merge fields settings edit page in their Mailchimp account.
No response
The text was updated successfully, but these errors were encountered:
Fixed bug by adding a conditional check for only public fields
public
Sorry, something went wrong.
MaxwellGarceau
Successfully merging a pull request may close this issue.
Describe the bug
In the WP Mailchimp settings, it's still possible to "include" a merge field that is has it's "visible" property unchecked in a Mailchimp account.
Note: The non visible merge fields will not display in the block editor or the front end
Solution
Replace the checkbox with "Not visible" text to inform the user that they must change the option in their account.
If we wanted to go the extra mile we could see if we could generate a link to that users merge fields settings edit page in their Mailchimp account.
Steps to Reproduce
Screenshots, screen recording, code snippet
non-visible-merge-fields-can-still-be-set-in-the-mailchimp-wp-admin.mov
Environment information
No response
WordPress information
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: