Skip to content

fix(caching): Take operation into account for cache prefix (#1365)#1366

Merged
klausi merged 1 commit intodrupal-graphql:8.x-4.xfrom
dulnan:fix/cache-prefix
Nov 10, 2023
Merged

fix(caching): Take operation into account for cache prefix (#1365)#1366
klausi merged 1 commit intodrupal-graphql:8.x-4.xfrom
dulnan:fix/cache-prefix

Conversation

@dulnan
Copy link
Copy Markdown
Contributor

@dulnan dulnan commented Sep 27, 2023

Fixes #1365.

@klausi
Copy link
Copy Markdown
Contributor

klausi commented Nov 10, 2023

Thanks, at first I did not understand that you can actually send different operation names with multiple queries.

Not sure why you would do that, as the graphql server will only execute one?

Anyway, I have a test case now that I will push in a follow-up PR and we can do this!

@klausi klausi merged commit 840f2c8 into drupal-graphql:8.x-4.x Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Caching does not take operation name into account

2 participants