- Issue created by @apmsooner
- Merge request !13343283: Refactor IncludeResolver to fix missing includes. β (Merged) created by apmsooner
- Status changed to Needs review
almost 2 years ago 5:42am 21 February 2023 - πΊπΈUnited States apmsooner
Revising patch to prevent normal route includes from working.
- First commit to issue fork.
-
albertosilva β
committed 2da611c6 on 1.0.x authored by
apmsooner β
Issue #3343283 by apmsooner: Only the first referenced paragraph is...
-
albertosilva β
committed 2da611c6 on 1.0.x authored by
apmsooner β
- πͺπΈSpain albertosilva Basque Country
HI,
Thanks for your patch, I've already merged it and will publish a new version soon.
- Assigned to albertosilva
- Status changed to Fixed
12 months ago 9:46am 29 December 2023 - πΊπΈUnited States apmsooner
Hey Alberto,
I ended up forking a version of your code as a sub-module into my own as I changed some other logic as well to get it to work the way i needed it to and its been quite a while since i submitted this patch. I've documented your module as an alternative. If you plan to continue supporting yours, be aware you will need to create a new version to support Drupal 10.2 as there are some breaking changes in jsonapi. You can see my latest version for reference: