- Issue created by @larowlan
- π¦πΊAustralia kim.pepper πββοΈπ¦πΊSydney, Australia
kim.pepper β made their first commit to this issueβs fork.
- Merge request !6621#3420991 Convert ImageToolkit plugin discovery to attributes β (Closed) created by kim.pepper
- Status changed to Needs review
7 months ago 10:57pm 15 February 2024 - π¦πΊAustralia kim.pepper πββοΈπ¦πΊSydney, Australia
Created a MR
- Status changed to RTBC
7 months ago 12:46am 16 February 2024 - π¦πΊAustralia mstrelan
Looks great. I think it's fine to omit the param docs for id and title. All ImageToolkit plugins have been converted to use annotations.
- Status changed to Needs work
7 months ago 3:23am 16 February 2024 - π¦πΊAustralia mstrelan
As per @larowlan I think we need to add
public readonly ?string $deriver = NULL
to the constructor params. - Status changed to Needs review
7 months ago 5:20am 16 February 2024 - π¦πΊAustralia kim.pepper πββοΈπ¦πΊSydney, Australia
Adds a deriver property
- Status changed to RTBC
7 months ago 7:57pm 17 February 2024 - πΊπΈUnited States smustgrave
Appears all feedback has been addressed.
Did a search for @ImageToolkit and all 4 instances of core have been replaced.
- Status changed to Fixed
6 months ago 11:48pm 28 February 2024 - π¬π§United Kingdom alexpott πͺπΊπ
Committed and pushed ffaf987160 to 11.x and e8b4e5c252 to 10.3.x. Thanks!
Adding to 3395575
-
alexpott β
committed d332de97 on 10.3.x
Issue #3420991 follow-uo by alexpott: Convert ImageToolkit plugin...
-
alexpott β
committed d332de97 on 10.3.x
-
alexpott β
committed e8b4e5c2 on 10.3.x
Issue #3420991 by kim.pepper, larowlan, mstrelan: Convert ImageToolkit...
-
alexpott β
committed e8b4e5c2 on 10.3.x
-
alexpott β
committed bd5aa8ff on 11.x
Issue #3420991 follow-uo by alexpott: Convert ImageToolkit plugin...
-
alexpott β
committed bd5aa8ff on 11.x
-
alexpott β
committed ffaf9871 on 11.x
Issue #3420991 by kim.pepper, larowlan, mstrelan: Convert ImageToolkit...
-
alexpott β
committed ffaf9871 on 11.x
- π¬π§United Kingdom alexpott πͺπΊπ
I committed a quick follow-up now that this, π Convert ImageToolkitOperation plugin discovery to attributes Needs review and π Convert ImageEffect plugin discovery to attributes RTBC are all in - in order to link the attributes together instead of pointing at the annotations. See https://git.drupalcode.org/project/drupal/-/commit/bd5aa8ffbe851e1a5104a...
Automatically closed - issue fixed for 2 weeks with no activity.