rajeshreeputra → changed the visibility of the branch 3552772-update-api-url to hidden.
@becw could you please try applying the changes from this MR, this should fix the issue you mentioned above point 8, 9.
Updated MR with proposed solution, requesting review.
Updated MR with following:
- Added requirement check in routing for point 1
- Updated config form to incorporate feedback for point 2 and 3.
After disconnecting a site, the authentication button appears twice. I believe it would be more effective not to display the Site Authentication button in first fieldset.
current:
Proposed:
2.x stable version is released. Hence closing this, feel free to reopen!
2.x stable version is released. Hence closing this, feel free to reopen!
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → made their first commit to this issue’s fork.
It appears this has already been addressed in version 4.x. Closing this for now, but please feel free to reopen if you notice any discrepancies.
Closing as we are unable to reproduce this issue. Please upgrade to the latest 4.1.1 release. If the issue persists, feel free to reopen.
Completed as part of 📌 Run Tests in Gitlab Parallelly Needs work .
rajeshreeputra → made their first commit to this issue’s fork.
@danflanagan8, update hook already present in 1.1.x release see.
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → changed the visibility of the branch 3551212-update-api-endpoints to hidden.
rajeshreeputra → made their first commit to this issue’s fork.
This issue is addressed as part of 🐛 Acquia DAM Item Update Queue not being ran Active , so closing this issue. If you continue to experience the problem, please feel free to reopen.
This looks good, but we may need to update it once ✨ Support Key module inside Acquia DAM (sites) Needs work lands. Adding this information to the authentication page can be addressed in conjunction with #3549173: Separate DAM Authentication Settings from Image Styles in Configuration Form → .
rajeshreeputra → made their first commit to this issue’s fork.
Updated CI template to run test in parallel, requesting review.
Change to Image Dimension Handling:
- The
getImageDimensions
method no longer immediately returns image property dimensions if found. Instead, it applies additional logic for original image formats. - For original images missing width/height data, a default size of
480x360
pixels is returned. - This change improves rendering robustness, especially when dealing with
SVG
or other images lacking explicit dimension metadata. - No behavioral change occurs for non-original image styles — dimension retrieval via image style configuration is unchanged.
Requesting review.
rajeshreeputra → made their first commit to this issue’s fork.
Status:
Unit
- Drupal 9 ✅
- Drupal 10 ✅
- Drupal 11.1 ✅
- Drupal 11.2 ✅
Kernel
- Drupal\Tests\acquia_dam\Kernel\ExpiredTagTest::testExpiryTag - This is already fixed in MR!188
Functional Javascript
Got warning, we'll investigate. Option --group cannot be used more than once
@cpierce42 Could you please share which version of Acquia DAM includes the “insert file as text with download” format, and provide a screenshot of how it appears? Also, if possible, a short video or GIF to showcase the functionality would be helpful.
I’ve investigated but have not been able to find this format in any version of Acquia DAM so far.
Validated changes in MR!197 and it fixes the items listed in #21 🐛 MultiFilter for checkbox and palette fields Active .
Please feel free to reopen the issue if you find it has not yet been fixed in 4.1.1 version.
rajeshreeputra → made their first commit to this issue’s fork.
rajeshreeputra → made their first commit to this issue’s fork.
This is fixed in 🐛 Unable to connect the site through Acquia Connector Active .
Yes, the response is cached from 10 hours in DAM.
Could you please try with clearing cache once?
Added Drupal 9, 10, 11 in CI:
- composer-drupal-core-9 runs for Drupal 9.
- Previous major runs fro Drupal 10.
- Current runs for Drupal 11 latest.
Next step:
- Look into phpunit parallel test execution.
Added Drupal 9, 10, 11 in CI:
- composer-drupal-core-9 runs for Drupal 9.
- Previous major runs fro Drupal 10.
- Current runs for Drupal 11 latest.
Next step:
- Look into phpunit parallel test execution.
rajeshreeputra → made their first commit to this issue’s fork.