- Merge request !7720Allow bulk operation forms to be used in a workspace. → (Closed) created by amateescu
- Status changed to Needs review
8 months ago 7:09am 25 April 2024 - 🇷🇴Romania amateescu
It turns out that the work done in 📌 Add an API for allowing modules to mark their forms as workspace-safe Needs work was pretty much on point and useful for the use cases tackled by this issue.
Some changes in this MR might seem weird, so I'll add some comments on the MR itself to try and explain them directly in a code context.
- 🇺🇸United States smustgrave
With the changes to view wonder if it needs sub-maintainer sign off? Thoughts?
- 🇷🇴Romania amateescu
Well.. the MR doesn't change any functionality in Views, so I don't think it's needed here :)
- Status changed to RTBC
8 months ago 8:44pm 28 April 2024 - 🇺🇸United States smustgrave
Ran the test-only feature and got the results here https://git.drupalcode.org/issue/drupal-2986005/-/jobs/1425066
Since workspace isn't marked stable in a released version yet assuming this is fine without a CR.
Reviewing the code believe this is good. Maybe future feature request or contrib module would be a UI to set these.
- 🇬🇧United Kingdom longwave UK
I think we should update the change record from the parent issue to mention this new trait and link to this issue.
- 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!
- Status changed to Fixed
8 months ago 10:26pm 29 April 2024