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

fix(app-headless-cms): pass default value to nested multiple values #4211

Merged
merged 3 commits into from
Jul 31, 2024

Conversation

leopuleo
Copy link
Contributor

Changes

With this PR, we are fixing a bug that occurs when setting defaultValues to a field with multiple values inside a dynamic zone template.

Steps to reproduce:

Dynamic zone + field with multiple values

  • Create a Dynamic Zone template with multiple values set to true.
  • Inside of it, add a text field with either of the two renderers, select-box or radio-buttons, with predefined values, one of them being selected: true.
  • Switch to Preview tab, the default value is not set.
  • If multiple values is set to false the default value is set correctly.

How Has This Been Tested?

Manually

@leopuleo leopuleo added this to the 5.40.4 milestone Jul 30, 2024
@leopuleo leopuleo requested a review from Pavel910 July 30, 2024 16:20
@leopuleo leopuleo self-assigned this Jul 30, 2024
@leopuleo leopuleo merged commit badd76f into dev Jul 31, 2024
92 checks passed
@leopuleo leopuleo deleted the leo/fix/multiple-nested-default-values branch August 5, 2024 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants