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

Set require: true on image block location field #6818

Open
wants to merge 4 commits into
base: develop-patch
Choose a base branch
from

Conversation

gitlabisbetterthangithub

Description

The location field in the image block wasn't required and editors could deselect the radio button options so no option was selected.

This pull request makes the location field required, thus requiring the field to always have an option selected

Ready?

  • In-code documentation (wherever needed)
  • Unit tests for fixed bug/feature
  • Tests and CI checks all pass

For review team

  • Add lab and/or sandbox examples (wherever helpful)
  • Add changes & docs to release notes draft in Notion

@gitlabisbetterthangithub
Copy link
Author

gitlabisbetterthangithub commented Nov 27, 2024

Somehow @bastianallgeier's other changes have been grouped into my pull request. It should only have been the one line of code I changed.

@gitlabisbetterthangithub gitlabisbetterthangithub changed the title Patch 1 Set require: true on image block location field Nov 27, 2024
@distantnative distantnative added this to the 4.6.0 milestone Dec 7, 2024
@distantnative
Copy link
Member

@gitlabisbetterthangithub if you rebase your PR with the current state from develop-patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants