Problem/Motivation

Requested by Razem
After #3163292: Have revisionable media entities in Varbase by default
No UI way to view changes or revert them
Change log by admin audit trail

Steps to reproduce

Given that we had enabled the "Create new revision" for the Image media type

Automatically create new revisions. Users with the "Administer media" permission will be able to override this option.

Proposed resolution

Add and enable the Media Revisions UI module with Varbase Media
https://www.drupal.org/project/media_revisions_ui

Media entities support revisions however there is no interface to handle them.

This module adds "Revisions" tab in the "Edit" section of media which displays a list of revisions and allows viewing, reverting or deleting revisions.

Revisions page for media item

#3165781: Add permissions for [Media Revisions UI] module
Permissions for media revisions UI
Permissions for media revisions UI

Remaining tasks

User interface changes

Have a UI tab when we edit media

CommentFileSizeAuthor
#15 Permissions--media-revistions-UI--dev-varbase8c1-media_library.png355.01 KBrajab natshah
#13 Revisions-for-Vardot-Logo-dev-varbase8c1-media_library.png69.18 KBrajab natshah
#6 Edit-Image-Vardot-Logo-dev-varbase8c1-media_library-with-revistion-tab.png86.18 KBrajab natshah
#6 Edit-Image-Vardot-Logo-dev-varbase8c1-media_library.png134.44 KBrajab natshah
#6 Admin-audit-trails-dev-varbase8c1-media_library.png40.82 KBrajab natshah

Comments

RajabNatshah created an issue. See original summary.

rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
Related issues: +#3163292: Have revisionable media entities in Varbase by default
rajab natshah’s picture

rajab natshah
he/him
commented
Title: Add [Media Revisions UI] module » Add [Media Revisions UI] module ~2.0

  • RajabNatshah committed 7093486 on 8.x-7.x 
    Issue #3163490: Add [Media Revisions UI] module ~2.0
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
StatusFileSize
new Admin-audit-trails-dev-varbase8c1-media_library.png40.82 KB
new Edit-Image-Vardot-Logo-dev-varbase8c1-media_library.png134.44 KB
new Edit-Image-Vardot-Logo-dev-varbase8c1-media_library-with-revistion-tab.png86.18 KB
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
Related issues: +#2911977: Add Media revision UI
rajab natshah’s picture

rajab natshah
he/him
commented
Related issues: +#2885486: Media entity is revisionable but doesn't have a revision link template
rajab natshah’s picture

rajab natshah
he/him
commented

The following is in core

Permission

view all media revisions:
  title: 'View all media revisions'
  description: 'To view a revision, you also need permission to view the media item.'

Route

entity.media.revision:
 path: '/media/{media}/revisions/{media_revision}/view'

But it can view any revision id in the URL for a selected media id.
It may show data for other media items or types too

mohammed j. razem’s picture

mohammed j. razem
Location Santa Clara, CA
commented

We will still need the module Media Revision UI so it can provide the UI to revert, and see the history of revisions.

As for the perms:
- Delete is not given to any role
- View is given to Editor upwards
- Revert is given to Content admin and Site admin only

This is similar to the content revisions.

rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
StatusFileSize
new Revisions-for-Vardot-Logo-dev-varbase8c1-media_library.png69.18 KB
rajab natshah’s picture

rajab natshah
he/him
commented
Related issues: +#3030911: Hiding 'revision_information' on a node form prevents the revision from being created
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
StatusFileSize
new Permissions--media-revistions-UI--dev-varbase8c1-media_library.png355.01 KB

  • RajabNatshah committed 2146123 on 8.x-7.x 
    Issue #3163490 by Mohammed J. Razem: Hide the revision but do not set to...
rajab natshah’s picture

rajab natshah
he/him
commented
Assigned: rajab natshah » mohammed j. razem
Status: Active » Needs review
Issue tags: +varbase-8.8.6, +varbase-9.0.0
rajab natshah’s picture

rajab natshah
he/him
commented
Assigned: mohammed j. razem » Unassigned

  • RajabNatshah committed 78ce6a1 on 8.x-7.x 
    Issue #3163490: Hide the revision but do not set to #access FALSE so new...
rajab natshah’s picture

rajab natshah
he/him
commented
Status: Needs review » Needs work

Needs to add the list of permissions in Varbase profile

rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes

#3165781: Add permissions for [Media Revisions UI] module

rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
Status: Needs work » Fixed
rajab natshah’s picture

rajab natshah
he/him
commented
Status: Fixed » Closed (fixed)
rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes