Migration Dashboard Enhancement

Created on 25 March 2025, about 1 month ago

Problem/Motivation

To simplify the migration process, we think it would be great to provide a three-step form:

Step 1: Configure โ€œMedia: Acquia DAMโ€ media types mapping:

  • Display the list of all Media: Acquia DAM media types only (excluding Acquia DAM Media Types), as this form is primarily meant to migrate source plugins of existing media types provided by the Media: Acquia DAM.
  • Listing of media types should be Sort by the number of media items (highest โ†’ lowest), so users can focus on relevant media types.
  • Allow configuration for only those media types with more than 0 media items. If a media type has 0 media items, we are enforcing those media types for deletion, instead of keeping an unused legacy Media: Acquia DAM media type.
  • Set Sync as the default method for handling media files, since their existing Media: Acquia DAM media types are already using this method only. Hence, we should default to this.
  • Auto-select the target source type if it matches our module's provided media types. (This is already working as expected today).
  • Restrict target source options based on media type relevance:
    • The target source types for Acquia DAM media types should be limited and it should be based on matching file extensions for each media type, so that we can limit / filter the options. For Ex: If Media Acquia DAM image media type supports png, jpeg file extensions only, then we should show only those Acquia DAM media types, which supports png, jpeg file extensions only.

Step 2: Configure โ€œAcquia DAMโ€ media types for deletion:

  • Users will see the list of all Media Types provided by Acquia DAM module with a delete checkbox next to each. They can opt to delete unnecessary media types during migration.
  • We are proposing Auto-select delete checkboxes, mainly for the following media types: Acquia DAM: Audio, Acquia DAM: Video, Acquia DAM: Image, and Acquia DAM: Document, since corresponding existing media types (provided by Media: Acquia DAM module) will exist post-migration for media types having media items more then 0. This helps avoid duplicates or redundant media types.
  • If a user unchecks an auto-selected delete option, a warning popup will appear, recommending deletion for clarity.
  • Newly added media types without a direct equivalent will not have their delete checkboxes auto-selected or prompt warnings upon unchecking.
  • A Back button will allow users to return to Step 1 for modifications and also Next Button for proceeding to Step 3.

Step 3: Migration Summary

  • Display a summary of migration configurations:
    • Which media types will be deleted.
    • Source Plugin mapping details.
    • Any label changes applied to media types (provided by Media: Acquia DAM module).
    • Method selected to handle media types (i.e Embed Or Sync method).
  • Provide:
    • Back button (to return to Step 2)
    • Save button (to store configuration changes for later migration).
    • Save & Migrate button (to apply changes and initiate migration immediately)
โœจ Feature request
Status

Active

Version

2.0

Component

Code

Created by

๐Ÿ‡ฎ๐Ÿ‡ณIndia vishalkhode

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @vishalkhode
  • Pipeline finished with Failed
    about 1 month ago
    Total: 291s
    #456953
  • Pipeline finished with Failed
    about 1 month ago
    Total: 222s
    #456985
  • First commit to issue fork.
  • Pipeline finished with Failed
    about 1 month ago
    Total: 226s
    #456992
  • Pipeline finished with Failed
    about 1 month ago
    Total: 358s
    #457029
  • Pipeline finished with Failed
    about 1 month ago
    Total: 359s
    #457086
  • Pipeline finished with Failed
    about 1 month ago
    Total: 296s
    #457098
  • Pipeline finished with Failed
    about 1 month ago
    Total: 205s
    #457110
  • Pipeline finished with Failed
    about 1 month ago
    Total: 254s
    #457162
  • Pipeline finished with Failed
    about 1 month ago
    Total: 210s
    #457248
  • Pipeline finished with Failed
    about 1 month ago
    Total: 542s
    #457308
  • Pipeline finished with Failed
    about 1 month ago
    Total: 213s
    #457636
  • Pipeline finished with Failed
    about 1 month ago
    Total: 256s
    #457790
  • Pipeline finished with Failed
    about 1 month ago
    Total: 519s
    #457805
  • Pipeline finished with Failed
    about 1 month ago
    Total: 626s
    #457847
  • Pipeline finished with Failed
    about 1 month ago
    Total: 245s
    #457853
  • Pipeline finished with Failed
    about 1 month ago
    Total: 190s
    #457864
  • Pipeline finished with Failed
    about 1 month ago
    Total: 222s
    #457957
  • Pipeline finished with Failed
    about 1 month ago
    Total: 560s
    #457990
  • Pipeline finished with Failed
    about 1 month ago
    Total: 555s
    #458470
  • Pipeline finished with Failed
    about 1 month ago
    Total: 196s
    #458510
  • Pipeline finished with Failed
    about 1 month ago
    #458529
  • Pipeline finished with Failed
    about 1 month ago
    Total: 345s
    #458925
  • Pipeline finished with Success
    about 1 month ago
    #463532
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia chandu7929 Pune

    Updated MR with required changes, hence requesting review.

  • Pipeline finished with Success
    about 1 month ago
    Total: 481s
    #464155
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ankitv18

    Please add a hook_update_N to by default enable the Acquia DAM module so that user can see the migration dashboard fully functional.

  • Pipeline finished with Success
    27 days ago
    Total: 562s
    #467172
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia chandu7929 Pune

    Since all requested changes are done, hence requesting review.

  • Pipeline finished with Success
    25 days ago
    Total: 10574s
    #468100
  • Pipeline finished with Success
    24 days ago
    #469182
  • Pipeline finished with Success
    24 days ago
    #469189
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    Need to continue reviewing code, but marking NW due to the following:
    Enable Acquia DAM and Media: Acquia DAM
    1) Add a few items to the Media: Acquia DAM image type
    2) Add a few items to the Acquia DAM image type
    3) Run the migration tool

    You'll notice that it suggests deleting the Acquia DAM: Image type, which contains existing data! Also, the media type labels are greyed out. The ones that come with Acquia DAM should be editable alongside the old Media: Acquia DAM types so you can get a full picture of the all the media types and their names.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia chandu7929 Pune

    @japerry - Thank you for highlighting first point; we will take it into consideration for inclusion. However, regarding your second point,

    Also, the media type labels are greyed out. The ones that come with Acquia DAM should be editable alongside the old Media: Acquia DAM types so you can get a full picture of the all the media types and their names.

    What is the reasoning behind permitting updates to the media labels of the new media type provided by the Acquia DAM module, despite the apparent logic in offering label modification options for the older media types?

  • Pipeline finished with Success
    18 days ago
    Total: 178s
    #474140
  • Pipeline finished with Failed
    18 days ago
    #474146
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    The media types that ship with Acquia DAM are fairly generic, so say you have a site with multiple image types due to permissions, download/sync, or another edge case that requires per media type delineation. You may then want to change the media type labels to reflect what they're used for, including the default one.

  • Pipeline finished with Failed
    18 days ago
    Total: 172s
    #474279
  • Pipeline finished with Failed
    17 days ago
    Total: 164s
    #475735
  • Pipeline finished with Failed
    16 days ago
    Total: 187s
    #476369
  • Pipeline finished with Failed
    13 days ago
    Total: 382s
    #478235
  • Pipeline finished with Failed
    13 days ago
    Total: 177s
    #478325
  • Pipeline finished with Failed
    12 days ago
    Total: 1402s
    #478378
  • Pipeline finished with Success
    12 days ago
    Total: 302s
    #478405
  • Pipeline finished with Failed
    12 days ago
    Total: 228s
    #478523
  • Pipeline finished with Success
    12 days ago
    Total: 309s
    #478545
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia chandu7929 Pune

    Incorporated requested changes, hence requesting review.

  • Pipeline finished with Failed
    11 days ago
    Total: 482s
    #479259
  • Pipeline finished with Failed
    11 days ago
    Total: 219s
    #479313
  • Pipeline finished with Failed
    10 days ago
    #480075
  • Pipeline finished with Failed
    10 days ago
    Total: 494s
    #480883
  • Pipeline finished with Failed
    10 days ago
    Total: 279s
    #480891
  • Pipeline finished with Failed
    10 days ago
    Total: 168s
    #480898
  • Pipeline finished with Success
    10 days ago
    Total: 273s
    #481027
  • Pipeline finished with Failed
    9 days ago
    Total: 294s
    #481156
  • Pipeline finished with Success
    9 days ago
    Total: 258s
    #481161
  • Pipeline finished with Success
    9 days ago
    Total: 184s
    #481231
  • Pipeline finished with Failed
    9 days ago
    Total: 241s
    #481712
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    Verified locally that indeed a test is failing: https://git.drupalcode.org/issue/media_acquiadam-3515234/-/jobs/5062463

    Failed asserting that the text of the element identified by '[data-drupal-selector="edit-media-label"] li:nth-child(2)' equals 'Acquia DAM Image โ†’ Media: Acquia DAM Image'.
    Failed asserting that two strings are equal.
    Expected :'Acquia DAM Image โ†’ Media: Acquia DAM Image'
    Actual :'Acquia DAM Video โ†’ Media: Acquia DAM Image'

    My guess is that the order is different on older versions of Drupal? nevertheless, we should have this fixed before committing.

  • Pipeline finished with Skipped
    3 days ago
    #486214
  • Pipeline finished with Skipped
    3 days ago
    #486215
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    Thanks for the test fixes! committed.

Production build 0.71.5 2024