Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
It is quite common for the workspaces to require additional metadata. Workspaces are content entities, so they support fields but there's no ui for managing them.
Proposed resolution
Add the field_ui tabs on /admin/config/workflow/workspaces
.
User interface changes
Field UI tabs showing up on the workspace listing page.
API changes
None.
Data model changes
None.
Release notes snippet
Workspaces now have the field ui tabs on the listing page.
Comment | File | Size | Author |
---|---|---|---|
#9 | 3054582-9.patch | 3 KB | jeqq |
#9 | interdiff.txt | 513 bytes | jeqq |
#7 | 3054582-7.patch | 3.01 KB | jeqq |
#7 | interdiff.txt | 1.94 KB | jeqq |
#2 | 3054582-2.patch | 943 bytes | blazey |
Comments
Comment #2
blazey CreditAttribution: blazey at Amazee Labs commentedComment #3
Leksat CreditAttribution: Leksat at Amazee Labs commentedTested manually. Looks good!
Also, the "Manage display" tab probably makes no sense. Or it does?
Comment #4
Gábor HojtsyI asked @amateescu about this, he wrote on slack:
Comment #5
blazey CreditAttribution: blazey at Amazee Labs commentedThanks for review. I think Manage display might make sense. A list of workspaces can be displayed with views for instance.
Comment #6
larowlanThis makes sense, but we need some tests here to make sure this functionality doesn't regress.
Thanks
Comment #7
jeqqAdded a test.
Comment #8
jeqqComment #9
jeqqModified the method comment.
Comment #10
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedThe test looks great, thanks!
Comment #11
Gábor HojtsySuperb!
Committed b759a93 and pushed to 8.8.x. Thanks!
Comment #13
Gábor HojtsyComment #14
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedThanks, @Gábor! This should be backportable to 8.7.x since Workspaces is still experimental :)
Comment #15
Manuel Garcia CreditAttribution: Manuel Garcia as a volunteer commentedPatch #9 applies cleanly to 8.7.x - queued the patch to test against that just now.
Comment #16
rosinegrean CreditAttribution: rosinegrean at PitechPlus commentedComment #17
catchCherry-picked to 8.7.x, thanks!
Comment #20
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedRemoving a tag since this was committed to 8.7.x too.