Follow-up for #3585354: add `no_cache: TRUE` to Canvas' Internal HTTP API routes where Dynamic Page Cache is less impactful
The following discussion from !1112 should be addressed: - [ ] @wotnak started a [discussion](https://git.drupalcode.org/project/canvas/-/merge_requests/1112#note_914652): (+3 comments) > I think we might also need to include the `parent` query arg in the cache contexts. Currently, when two reference fields point to the same entity type and bundle, the first response is cached and reused for subsequent calls with a different `?parent=` and the property expressions in the reused response include the parent from the first request instead of from the current request. - [ ] @wimleers: > Wouldn't it be a better use of server resources _and_ our time as Canvas developers to specify `no_cache: TRUE` to trigger `\Drupal\Core\PageCache\ResponsePolicy\DenyNoCacheRoutes` for Dynamic Page Cache? - [ ] @penyaskito: > let's add `'url.query_args:parent'` cache tag, and consider a follow-up for this for every Canvas route that might benefit from it This is that follow-up.
issue