Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return GraphQL response as GdsApi::Response #1304

Merged
merged 2 commits into from
Dec 3, 2024
Merged

Conversation

brucebolt
Copy link
Member

This is instead of Hash.

In Government Frontend, we need some other information from the request response (e.g. the cache control headers) which are not included when the response is converted to a hash.

This is not a breaking change as the new object type accepts the same methods as the hash.

Trello card

This is instead of `Hash`.

In Government Frontend, we need some other information from the request
response (e.g. the cache control headers) which are not included when
the response is converted to a hash.

This is not a breaking change as the new object type accepts the same
methods as the hash.
@brucebolt brucebolt marked this pull request as ready for review December 2, 2024 17:05
@brucebolt brucebolt merged commit 600797b into main Dec 3, 2024
41 checks passed
@brucebolt brucebolt deleted the graphql-response branch December 3, 2024 08:00
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.

2 participants