- Issue created by @alex.skrypnyk
- Assigned to joshua1234511
- Assigned to alex.skrypnyk
- Status changed to Needs review
11 months ago 6:03pm 23 January 2024 - 🇮🇳India joshua1234511 Goa
PR raised: https://github.com/civictheme/monorepo-drupal/pull/1206
Implemented:
- Created Media tags (civictheme_media_tags) vocabulary.
- Added permissions for the Content author and Site administrator roles to perform CRUD operations on terms (but not the vocabulary).
- Added Behat tests for permissions.
- Added entity reference Media tags ( field_m_media_tags ) field to each media entity.
- Updated existing Behat tests for fields on media entities.
- Created a clone of vanilla Drupal Media view and store it as optional config in theme (named with civictheme prefix ). Adjust filters to show the Media tags filter and allow to match on partial tag matches.
- Implemented an additional settings provisioning option to replace Media view with the one shipped in CT configuration.
- Added a Behat test that would be checking that a Content->Media page has the required filters and can be searched through.
Not Implemented/ Questions
- Create a clone of vanilla Drupal Media library view and store it as optional config in theme (leave the name as-is). Adjust filters to show the Media tags filter and allow to match on partial tag matches.
- Cloning is not enough, there are theme template and preprocessing required to obtain the look na feel of existing view
- https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/media...
- https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/media...
- https://git.drupalcode.org/project/drupal/-/tree/11.x/core/modules/media...
- One way can be update the exiting view add it to optional and load as single import via content provisions.
- c67bed06 committed on 1.x
Issue #3415737 by joshua1234511: Added provision for Media tags...
- c67bed06 committed on 1.x
- Assigned to sonam.chaturvedi
- Status changed to Fixed
10 months ago 2:04am 7 March 2024 - 🇦🇺Australia alex.skrypnyk Melbourne
Tested and implemented. Will be released in 1.7
Automatically closed - issue fixed for 2 weeks with no activity.