Skip to content

fix(ui): skip admin.condition checks in bulk edit#16387

Open
harshdhankhar11 wants to merge 1 commit intopayloadcms:mainfrom
harshdhankhar11:fix/admin-condition-bulk-edit-16336
Open

fix(ui): skip admin.condition checks in bulk edit#16387
harshdhankhar11 wants to merge 1 commit intopayloadcms:mainfrom
harshdhankhar11:fix/admin-condition-bulk-edit-16336

Conversation

@harshdhankhar11
Copy link
Copy Markdown

Closes #16336

Summary

This PR fixes an issue where fields with admin.condition appear in the bulk edit field selector but are silently hidden after selection.

Problem

In the bulk edit flow, selected fields can be rendered without meaningful sibling document state. Because of that, admin.condition may evaluate to false, which hides fields that the user explicitly selected for bulk editing.

Fix

  • Skip admin.condition checks in the bulk edit flow
  • Preserve normal admin.condition behavior outside bulk edit
  • Ensure explicitly selected bulk edit fields render as expected

Testing

@harshdhankhar11
Copy link
Copy Markdown
Author

Opened this PR for #16336. I’ve also added regression coverage for the bulk edit flow.

@harshdhankhar11
Copy link
Copy Markdown
Author

Hey folks @JarrodMFlesch @jacobsfletch @AlessioGr , I put up a fix for the bulk-edit admin.condition bug (#16387) with test coverage. Just waiting on a workflow approval to get the CI green!"

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.

Fields with admin.condition are silently hidden in bulk edit

1 participant