Pune
Account created on 21 March 2016, over 9 years ago
#

Merge Requests

More

Recent comments

🇮🇳India rajeshreeputra Pune

Fixed failing test, requesting review.

🇮🇳India rajeshreeputra Pune

rajeshreeputra changed the visibility of the branch 3552772-update-api-url to hidden.

🇮🇳India rajeshreeputra Pune

@becw could you please try applying the changes from this MR, this should fix the issue you mentioned above point 8, 9.

🇮🇳India rajeshreeputra Pune

Re run the failed job, it should be green now.

🇮🇳India rajeshreeputra Pune

Updated MR with proposed solution, requesting review.

🇮🇳India rajeshreeputra Pune

Updated MR with following:

  1. Added requirement check in routing for point 1
  2. 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:

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

2.x stable version is released. Hence closing this, feel free to reopen!

🇮🇳India rajeshreeputra Pune

2.x stable version is released. Hence closing this, feel free to reopen!

🇮🇳India rajeshreeputra Pune

Released 2.3.0!

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

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.

🇮🇳India rajeshreeputra Pune

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.

🇮🇳India rajeshreeputra Pune

Completed as part of 📌 Run Tests in Gitlab Parallelly Needs work .

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

@danflanagan8, update hook already present in 1.1.x release see.

🇮🇳India rajeshreeputra Pune

Reviewed and validated changes looks good.

🇮🇳India rajeshreeputra Pune

Resolved conflict and rebased.

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

rajeshreeputra changed the visibility of the branch 3551212-update-api-endpoints to hidden.

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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.

🇮🇳India rajeshreeputra Pune

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 .

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

ohhh, yes, updated.

🇮🇳India rajeshreeputra Pune

@becw, you are right.

After thoroughly reviewing the processQueue() method, I found that this code causes the processing to exit early, so it never reaches the code further down that actually unpublishes the asset/media.

Will validate this one more time.

🇮🇳India rajeshreeputra Pune

Updated CI template to run test in parallel, requesting review.

🇮🇳India rajeshreeputra Pune

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.

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

@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.

🇮🇳India rajeshreeputra Pune

Validated changes in MR!197 and it fixes the items listed in #21 🐛 MultiFilter for checkbox and palette fields Active .

🇮🇳India rajeshreeputra Pune

Please review.

🇮🇳India rajeshreeputra Pune

This is not required.

🇮🇳India rajeshreeputra Pune

Please feel free to reopen the issue if you find it has not yet been fixed in 4.1.1 version.

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

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

🇮🇳India rajeshreeputra Pune

8.x-1.8 is the new tag.

🇮🇳India rajeshreeputra Pune

Please review.

🇮🇳India rajeshreeputra Pune

Please review.

🇮🇳India rajeshreeputra Pune

Requesting review.

🇮🇳India rajeshreeputra Pune

Yes, the response is cached from 10 hours in DAM.

Could you please try with clearing cache once?

🇮🇳India rajeshreeputra Pune

Added Drupal 9, 10, 11 in CI:

  1. composer-drupal-core-9 runs for Drupal 9.
  2. Previous major runs fro Drupal 10.
  3. Current runs for Drupal 11 latest.

Next step:

  1. Look into phpunit parallel test execution.
🇮🇳India rajeshreeputra Pune

Added Drupal 9, 10, 11 in CI:

  1. composer-drupal-core-9 runs for Drupal 9.
  2. Previous major runs fro Drupal 10.
  3. Current runs for Drupal 11 latest.

Next step:

  1. Look into phpunit parallel test execution.
🇮🇳India rajeshreeputra Pune

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

Production build 0.71.5 2024