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: change file-control default value to string #6934

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

demshy
Copy link
Member

@demshy demshy commented Oct 10, 2023

Fix for issue #6918

Since recent changes to image gallery & sorting, file widget value defaulted to an empty array, which caused incorrect structure when single field was used within list. This reverts to how it was before 3.0.

@netlify
Copy link

netlify bot commented Oct 10, 2023

Deploy Preview for cms-demo ready!

Name Link
🔨 Latest commit cb25cbb
🔍 Latest deploy log https://app.netlify.com/sites/cms-demo/deploys/6525474c462859000827fcdc
😎 Deploy Preview https://deploy-preview-6934--cms-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the type: bug code to address defects in shipped code label Oct 10, 2023
@demshy demshy merged commit 90c9844 into master Oct 10, 2023
20 checks passed
@demshy demshy deleted the fix/file-control-single-field branch October 10, 2023 14:31
martinjagodic pushed a commit to geotrev/netlify-cms that referenced this pull request Oct 17, 2023
@tizzle
Copy link

tizzle commented May 21, 2024

I think this now leads to new problems as you can see on #7120 and #7186.

Imho the entry should be removed completely and not set to an empty string.

@martinjagodic as you approved the changes, could you please take care of #7120 and #7186?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug code to address defects in shipped code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants