- Issue created by @capysara
- Merge request !68#3471261 β Update error log message for integration link removal β (Open) created by capysara
- Issue was unassigned.
- Status changed to Needs review
7 months ago 3:28pm 30 August 2024 - πΊπΈUnited States capysara
Created a MR to update the log error to be more helpful. I need to do some more research because this issue might part of a larger problem or might be specific to my site, but the error message is a positive step regardless.
- Status changed to Needs work
5 months ago 2:36pm 5 November 2024 - ππΊHungary Balu Ertl Budapest πͺπΊ
MRs of bugfixes should target the
1.0.x
branch. - Merge request !84#3471261 β Update error log message for integration link removal β (Merged) created by capysara
- πΊπΈUnited States capysara
capysara β changed the visibility of the branch asset-id-missing-integration-link-3471261 to hidden.
- πΊπΈUnited States capysara
capysara β changed the visibility of the branch 3471261-missing-integration-links to hidden.
- First commit to issue fork.
-
japerry β
committed cdb8d840 on 1.0.x authored by
capysara β
#3471261 β Update error log message for integration link removal
-
japerry β
committed cdb8d840 on 1.0.x authored by
capysara β
-
japerry β
committed 23187da2 on 1.1.x authored by
capysara β
#3471261 β Update error log message for integration link removal
-
japerry β
committed 23187da2 on 1.1.x authored by
capysara β
- ππΊHungary Balu Ertl Budapest πͺπΊ
On Acquia's internal channels, we have received reports of very similar errors to happen. Based on the formatting of error messages I see on Watchdog screenshots, I can tell that the Customer was using a version of this module before the MR !84 was merged.
Although handling of HTTP 4xx errors from asset-related API requests has been improved in the 1.1.x branch, but it is uncertain whether it covers integration link-related requests too or not. Independently from HTTP 4xx errors, I do agree with @capysara β 's suspicion about if the CRUD operations of integration links are not in full sync with the DAM assets during their entire lifecycle. As the auto-hiding logic instantly sets media items unpublished at any time when the Widen API returns a response with negative meaning (aka. asset unavailable or deleted), these are not permanent changes on Widen's side (asset can be available later on and even can be restored from trashbin by admin). Therefore we don't delete media items automatically, so β up to my current understanding β we should not delete integration links either.
For the sake of transparency (and because we have referenced this ticket earlier from various sources) I restored its original issue description which was clearly about solving the link deletion problems. Also, bumping branch up to
1.1.x
because it could be easier due to the aforementioned feature already implemented there.