Problem/Motivation

Update the CanvasHooks::acquiaDamStoragePropShapeAlter() method to robustly handle multiple Acquia DAM media types.

Update and document the handling of Acquia DAM media types so that new types and field mappings can be added or updated easily.

This will help the further DAM media support in canvas.

Issue fork acquia_dam-3551419

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

ankitv18 created an issue. See original summary.

ankitv18 opened merge request !205

vishalkhode made their first commit to this issue’s fork.

vishalkhode’s picture

vishalkhode commented
Title: Refactor canvasHooks for further DAM media types support » Enhance acquiaDamStoragePropShapeAlter() method for further DAM media support in Canvas
Issue summary: View changes
Status: Active » Needs review
Parent issue: » #3519247: Acquia DAM and Canvas integration

japerry made their first commit to this issue’s fork.

rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Status: Needs review » Reviewed & tested by the community

Changes looks good to me, RTBC.

rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Status: Reviewed & tested by the community » Needs work

Requested changes, moving in NW.

ankitv18’s picture

ankitv18 commented
Status: Needs work » Needs review
rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Status: Needs review » Reviewed & tested by the community

Changes looks good.

amangrover90 made their first commit to this issue’s fork.

amangrover90’s picture

amangrover90 commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.