When running cron the first time after install we get the following error:

Failed to fetch asset ids: Client error: `GET https://api.widencollective.com/v2/assets/search?limit=100&offset=10000&query=lastEditDate%3A%5Bafter%201969-12-31T17%3A00%3A00Z%5D%20AND%20rd%3A%28%5Bbefore%2008%2F11%2F2022%5D%20OR%20%5B08%2F11%2F2022%5D%29%20AND%20ed%3A%28%28isEmpty%29%20OR%20%5Bafter%2008%2F11%2F2022%5D%29&include_deleted=true&include_archived=true` resulted in a `400 Bad Request` response:

{
  "error": true,
  "response_code": 400,
  "error_message": "Sum of limit and offset must not exceed 10,000.",
  "stac (truncated...)

Causes For Concern
What impact does this have on accounts with more than 10,000 assets?
Can this happen other times than first cron run and what would be the consiquences?

Steps to Reproduce
1. Install module
2. Connect to DAM account with more than 10,000 assets
3. Run cron

Issue fork acquia_dam-3303359

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:

Comments

damondt created an issue. See original summary.

damondt’s picture

Title: API Error on Cron "Sum of limit and offset must not exceed 10,000." » API Error on first cron run "Sum of limit and offset must not exceed 10,000."
Issue summary: View changes
damondt’s picture

Issue summary: View changes
damondt’s picture

Issue summary: View changes
japerry’s picture

Project: Media: Acquia DAM » Acquia DAM
Version: 2.0.5 » 1.1.x-dev

Moving to the Acquia DAM queue as I'm guessing its still an issue? If not I'd suggest upgrading (if you haven't already)

vipin.mittal18’s picture

Status: Active » Postponed (maintainer needs more info)
manojapare’s picture

StatusFileSize
new201.64 KB
new217.67 KB

Issue is reproduce able by clicking on Last in pagination of Media Library form field widget.
Screenshots:
1. Before clicking the Last pagination link (Hided the actual image assets from the screenshot) Before clicking the Last pagination link
2. After clicking the Last pagination link After clicking the Last pagination link

vipin.mittal18’s picture

Status: Postponed (maintainer needs more info) » Needs work

Thanks @manojapare, we are actively implementing the necessary fixes, and you can expect a prompt resolution to this concern.

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

ankitv18’s picture

Added a minor suggestion to shorter warning message, rest looks good and resolve the problem.

Patch: https://git.drupalcode.org/project/acquia_dam/-/merge_requests/103.patch (Use this patch for now)

vipin.mittal18’s picture

Status: Needs work » Needs review
ankitv18’s picture

StatusFileSize
new84.96 KB

Thats how the warning message looks like ~~ included the patch to validate on local
warning

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

rajeshreeputra’s picture

Status: Needs review » Fixed

For some reason, this issue remains in the review state. However, I am marking it as fixed.
sorry wrong comment.

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

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

Maintainers, please credit people who helped resolve this issue.

rajeshreeputra’s picture

Status: Fixed » Needs review

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

amangrover90’s picture

Status: Needs review » 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.