- Issue created by @agentrickard
- Status changed to Needs review
8 months ago 4:40pm 22 April 2024 - 🇺🇸United States japerry KVUO
Ohh, thanks for catching that Ken! I've escalated it internally so we can get it fixed in both 1.0 and the upcoming 1.1 release.
- last update
8 months ago Composer require-dev failure - 🇺🇸United States japerry KVUO
Looking at views_remote_data as well -- since I'm guessing any derivative plugin will have this issue as well.
- 🇺🇸United States agentrickard Georgia (US)
Maybe -- though I haven't run into that yet.
- Status changed to Needs work
8 months ago 8:29pm 2 May 2024 - 🇺🇸United States japerry KVUO
Okay I spent some time on this today, and this issue is a doozy.
1) This -will- break 10.2 and earlier, unless a BC shim is added. This will be necessary before committing.
2) The tests (automated and manual) are failing because of the change. Interestingly, the dev tests say they are passing, but if you look at the debug output they are also failing for 10.3.
3) I'm a little leary of using the uuid from the asset as the mid because a) there is an associated mid after it is created and b) not sure how non-numeric indexes work here.So I'm a bit worried about how we handle this missing media id in the view. Will try to get Matt's time to look at it sometime later in May, but before Drupal 10.3 comes out.
- First commit to issue fork.
- 🇺🇸United States mglaman WI, USA
Cherry-picked commit from https://git.drupalcode.org/project/acquia_dam/-/merge_requests/47 which fixes the mock handler and should clear up some errors.
-
japerry →
committed 0da7950e on 1.0.x authored by
Rajeshreeputra →
Issue #3442662: Incompatible with Drupal 10.3 and higher
-
japerry →
committed 0da7950e on 1.0.x authored by
Rajeshreeputra →
- Status changed to Fixed
7 months ago 8:13pm 7 June 2024 -
japerry →
committed 58ceedf3 on 1.1.x authored by
Rajeshreeputra →
Issue #3442662: Incompatible with Drupal 10.3 and higher
-
japerry →
committed 58ceedf3 on 1.1.x authored by
Rajeshreeputra →
Automatically closed - issue fixed for 2 weeks with no activity.