- Status changed to Needs review
over 1 year ago 1:41pm 10 August 2023 - last update
over 1 year ago 29,958 pass - 🇮🇳India sourabhjain
I have updated the changes mentioned in #4. Please review.
- Status changed to RTBC
over 1 year ago 6:03pm 16 August 2023 - 🇺🇸United States smustgrave
#4 does appear to have been copied over.
RTBC+1
- last update
over 1 year ago 29,961 pass - last update
over 1 year ago 30,047 pass - last update
over 1 year ago 30,052 pass - last update
over 1 year ago 30,056 pass - last update
over 1 year ago 30,058 pass - last update
over 1 year ago 30,060 pass - last update
over 1 year ago 30,060 pass 35:58 33:55 Running- Status changed to Needs work
over 1 year ago 4:17am 31 August 2023 - 🇳🇿New Zealand quietone
I'm triaging RTBC issues → . I re-read the IS and the comments.
My comment in #18 was a simple read of the suggested text, I did not check it for accuracy. In hindsight, that looks like a mistake.
It is a 9 year old comment and it does need to read and checked for accuracy. I did a bit of that by searching for each interface or method referred to in the comment. It turns out that \Drupal\Component\Plugin\ConfigurablePluginInterface no longer exists. It was removed in #3081145: Remove BC layers in the Plugin component → .
Setting back to Needs work and removing the novice tag.
- Merge request !8757Documentation in ConfigDependencyManager conflates plugin dependencies and config dependencies → (Closed) created by quietone
- Status changed to Needs review
5 months ago 8:43am 14 July 2024 - 🇳🇿New Zealand quietone
Converted to MR and updated the text for a removed interface.
- Status changed to RTBC
5 months ago 1:07pm 17 July 2024 - 🇺🇸United States smustgrave
"Entities with plugin dependencies need to" went right up to that 80 character line haha
Update seems good to me
- 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked back through to 10.3.x, thanks!
- Status changed to Fixed
5 months ago 9:43am 29 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.