[PR #3553] [MERGED] refactor: interceptor error display in graphql response #4427

Closed
opened 2026-03-17 01:57:56 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/3553
Author: @nivedin
Created: 11/15/2023
Status: Merged
Merged: 11/17/2023
Merged by: @AndrewBastin

Base: release/2023.12.0Head: refactor/interceptor-error-display-graphql


📝 Commits (3)

  • 72aef07 fix: graphQL interceptor error
  • 9ab4545 fix: add error placeholder when sending req
  • 273f92a chore: pass i18n to function and remove toast

📊 Changes

6 files changed (+118 additions, -24 deletions)

View changed files

📝 packages/hoppscotch-common/src/components.d.ts (+6 -1)
📝 packages/hoppscotch-common/src/components/graphql/RequestOptions.vue (+25 -7)
📝 packages/hoppscotch-common/src/components/graphql/Response.vue (+24 -4)
📝 packages/hoppscotch-common/src/components/graphql/SubscriptionLog.vue (+5 -1)
📝 packages/hoppscotch-common/src/helpers/graphql/connection.ts (+57 -10)
📝 packages/hoppscotch-common/src/platform/std/interceptors/extension.ts (+1 -1)

📄 Description

Closes HFE-316

Description

This PR adds an error placeholder in the response section of GraphQl if there is an error occured due to not adding the URL in the extension origin list.

image

Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

Additional Info

  • Same was added for REST section in this PR

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/3553 **Author:** [@nivedin](https://github.com/nivedin) **Created:** 11/15/2023 **Status:** ✅ Merged **Merged:** 11/17/2023 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `release/2023.12.0` ← **Head:** `refactor/interceptor-error-display-graphql` --- ### 📝 Commits (3) - [`72aef07`](https://github.com/hoppscotch/hoppscotch/commit/72aef07573ad288678309bf54789e754efd5f082) fix: graphQL interceptor error - [`9ab4545`](https://github.com/hoppscotch/hoppscotch/commit/9ab4545769c9bafe89ed06a3bc0c6c258a2ed79b) fix: add error placeholder when sending req - [`273f92a`](https://github.com/hoppscotch/hoppscotch/commit/273f92ac0235645f6ba6623c823df04673fe41ff) chore: pass i18n to function and remove toast ### 📊 Changes **6 files changed** (+118 additions, -24 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-common/src/components.d.ts` (+6 -1) 📝 `packages/hoppscotch-common/src/components/graphql/RequestOptions.vue` (+25 -7) 📝 `packages/hoppscotch-common/src/components/graphql/Response.vue` (+24 -4) 📝 `packages/hoppscotch-common/src/components/graphql/SubscriptionLog.vue` (+5 -1) 📝 `packages/hoppscotch-common/src/helpers/graphql/connection.ts` (+57 -10) 📝 `packages/hoppscotch-common/src/platform/std/interceptors/extension.ts` (+1 -1) </details> ### 📄 Description Closes HFE-316 ### Description This PR adds an error placeholder in the response section of GraphQl if there is an error occured due to not adding the URL in the extension origin list. ![image](https://github.com/hoppscotch/hoppscotch/assets/53208152/7caafb9d-e050-4b96-a603-34c4dbb50353) ### Checks - [x] My pull request adheres to the code style of this project - [ ] My code requires changes to the documentation - [ ] I have updated the documentation as required - [x] All the tests have passed ### Additional Info - Same was added for REST section in this [PR](https://github.com/hoppscotch/hoppscotch/pull/3548) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 01:57:56 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hoppscotch#4427
No description provided.