Skip to content

feat(app-bridge-app): add executeGraphQlWithFullResponse and deprecate executeGraphQl#1596

Merged
SedukhinaPolina merged 3 commits into
mainfrom
fix/de-123-add-errors-to-executeGraphQl-response
Jun 11, 2026
Merged

feat(app-bridge-app): add executeGraphQlWithFullResponse and deprecate executeGraphQl#1596
SedukhinaPolina merged 3 commits into
mainfrom
fix/de-123-add-errors-to-executeGraphQl-response

Conversation

@SedukhinaPolina

@SedukhinaPolina SedukhinaPolina commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

executeGraphQl resolves only the data field of the GraphQL response and the top-level errors array is silently dropped, so consuming apps can't see GraphQL errors (which arrive at HTTP 200). executeGraphQlWithFullResponse resolves the full response envelope instead.

The host side (web-app) must return the full envelope for the new method - the PR comes separately.

Linear

@SedukhinaPolina SedukhinaPolina requested a review from julianiff June 5, 2026 13:57
@SedukhinaPolina SedukhinaPolina requested a review from a team as a code owner June 5, 2026 13:57
@changeset-bot

changeset-bot Bot commented Jun 5, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 215d241

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@frontify/app-bridge-app Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@SedukhinaPolina SedukhinaPolina changed the title feat(app-bridge-app): add executeGraphQlWithErrors and deprecate executeGraphQl feat(app-bridge-app): add executeGraphQlWithFullResponse and deprecate executeGraphQl Jun 5, 2026
@SedukhinaPolina SedukhinaPolina merged commit 2d76186 into main Jun 11, 2026
28 checks passed
@SedukhinaPolina SedukhinaPolina deleted the fix/de-123-add-errors-to-executeGraphQl-response branch June 11, 2026 08:04
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