Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

allow editors to remove a fieldset in easyform #447

Closed
wants to merge 1 commit into from

Conversation

gotcha
Copy link
Member

@gotcha gotcha commented Jan 13, 2025

context: this permission wasn't set since the delete-fieldset option was added to plone.schemaeditor

context: this permission wasn't set since the delete-fieldset option
was added to plone.schemaeditor
@gotcha gotcha force-pushed the gotcha/enable-remove-fieldset branch from 8f49118 to 0dd5292 Compare January 13, 2025 15:27
@gotcha gotcha requested review from thet and ale-rt January 13, 2025 15:35
Copy link
Member

@ale-rt ale-rt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I did not really test it.

@gotcha
Copy link
Member Author

gotcha commented Jan 16, 2025

merged manually

@gotcha gotcha closed this Jan 16, 2025
@gotcha gotcha deleted the gotcha/enable-remove-fieldset branch January 16, 2025 13:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants