DAM video not displaying when download and sync is enabled

Created on 21 January 2025, 2 months ago

Problem/Motivation

When Download and Sync options are enabled for videos in the DAM system, the video fails to display properly. Users may encounter a blank screen, even though the file appears to be synced and downloaded successfully.

Steps to reproduce

  • create a new project using drupal-recommended-project
  • enable acquia dam and enter all configurations required
  • enable Download and Sync assets at admin/structure/media/manage/acquia_dam_video_asset and save configuration
  • create a DAM media reference field in page/article content type and select Acquia DAM Video and save configuration
  • create a new content and select any video in media field and save content
  • visit the node page and you will see DAM videos are not showing

Look at the screenshot below

๐Ÿ› Bug report
Status

Needs work

Version

1.1

Component

Code

Created by

๐Ÿ‡ฎ๐Ÿ‡ณIndia deepakkm

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

Merge Requests

Comments & Activities

  • Issue created by @deepakkm
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia rajeshreeputra Pune

    The functionality to display/play local video media is currently unavailable in the Acquia DAM interface. This feature is present in the Media: Acquia DAM, this need to be updated accordingly for consistency and enhanced user experience.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ankitv18

    ankitv18 โ†’ changed the visibility of the branch 3501141-dam-video-formatter to hidden.

  • Pipeline finished with Canceled
    2 months ago
    Total: 88s
    #405621
  • Pipeline finished with Canceled
    2 months ago
    Total: 171s
    #405637
  • Pipeline finished with Success
    2 months ago
    Total: 2191s
    #405640
  • Pipeline finished with Success
    2 months ago
    Total: 2051s
    #407314
  • Pipeline finished with Canceled
    2 months ago
    Total: 306s
    #407359
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ankitv18
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ankitv18

    MR!115 can be reviewed now!!

  • Pipeline finished with Success
    2 months ago
    Total: 2040s
    #407363
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia deepakkm

    Lets update the MR and use this MR.

  • Pipeline finished with Success
    2 months ago
    Total: 2276s
    #408902
  • Pipeline finished with Success
    2 months ago
    Total: 2307s
    #408937
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia deepakkm

    this has been taken care in the latest commit as we have handled the situation on the basis of media source now.

    If you attempt to create a new media type from Acquia DAM with source Acquia DAM: Video and named "DAM Remote Video" with the machine name "dam_remote_video" it fail to work as expected. This is because the bundle name doesn't align with the predefined one, "acquia_dam_video_asset".

  • Pipeline finished with Failed
    2 months ago
    Total: 2349s
    #408993
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia vishalkhode

    vishalkhode โ†’ made their first commit to this issueโ€™s fork.

  • Pipeline finished with Success
    2 months ago
    Total: 2240s
    #410196
  • Pipeline finished with Success
    2 months ago
    Total: 2096s
    #410241
  • Pipeline finished with Failed
    about 2 months ago
    Total: 345s
    #414596
  • Pipeline finished with Failed
    about 2 months ago
    Total: 572s
    #414600
  • Pipeline finished with Failed
    about 2 months ago
    Total: 605s
    #414615
  • Pipeline finished with Failed
    about 2 months ago
    Total: 462s
    #414620
  • Pipeline finished with Failed
    about 2 months ago
    Total: 751s
    #414628
  • Pipeline finished with Failed
    about 2 months ago
    Total: 764s
    #414638
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2105s
    #414678
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2150s
    #414743
  • Pipeline finished with Success
    about 2 months ago
    Total: 2104s
    #414789
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ankitv18

    Changes are updated and pipelines are passing.

  • Pipeline finished with Canceled
    about 2 months ago
    Total: 335s
    #419895
  • Pipeline finished with Canceled
    about 2 months ago
    Total: 457s
    #419900
  • Pipeline finished with Success
    about 2 months ago
    Total: 2065s
    #419902
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    Made comments in the MR. it needs a bit of work.

  • Pipeline finished with Canceled
    about 2 months ago
    Total: 607s
    #421009
  • Pipeline finished with Canceled
    about 2 months ago
    Total: 269s
    #421028
  • Pipeline finished with Canceled
    about 2 months ago
    Total: 444s
    #421036
  • Pipeline finished with Canceled
    about 2 months ago
    Total: 225s
    #421041
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2108s
    #421047
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2409s
    #421124
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2165s
    #421218
  • Pipeline finished with Canceled
    about 2 months ago
    Total: 2025s
    #421306
  • Pipeline finished with Canceled
    about 2 months ago
    Total: 158s
    #421327
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2102s
    #421329
  • Pipeline finished with Canceled
    about 2 months ago
    #421355
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2217s
    #421356
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2382s
    #421640
  • Pipeline finished with Canceled
    about 2 months ago
    Total: 234s
    #421663
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2409s
    #421664
  • Pipeline finished with Failed
    about 2 months ago
    Total: 2342s
    #421856
  • Pipeline finished with Success
    about 2 months ago
    Total: 2012s
    #422000
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    This needs a kernel test to validate functionality.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ankitv18

    I've incorporate the changes with https://git.drupalcode.org/project/acquia_dam/-/merge_requests/113/diffs... (as this MR consists of Field swapping on media_type_presave hook with most of the logic using Asset, AssetDriver and Hook)

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO
Production build 0.71.5 2024