[PR #3466] [MERGED] feat(common): display status text from the API response if available #4393

New issue

Closed

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

kerem commented

2026-03-17 01:56:01 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/3466
Author: @kpguru20001
Created: 10/21/2023
Status: ✅ Merged
Merged: 12/4/2023
Merged by: @AndrewBastin

Base: release/2023.12.0 ← Head: feature/display-statusText-#3458

📝 Commits (4)

4e2bf1f docs(Common): 📝 an RFC document to explain the PR and its relevant changes
9799b7b feat(Common): ✨ pass statusText from axios res to hoppRestResponse and display
bca368a fix(PR-Comments): 🐛 removed console logs and dead code files
48dd94e refactor: eliminate redundancy

📊 Changes

5 files changed (+25 additions, -4 deletions)

View changed files

📝 packages/hoppscotch-common/src/components/http/ResponseMeta.vue (+6 -1)
📝 packages/hoppscotch-common/src/helpers/network.ts (+1 -1)
📝 packages/hoppscotch-common/src/helpers/types/HoppRESTResponse.ts (+2 -1)
📝 packages/hoppscotch-common/src/helpers/utils/statusCodes.ts (+12 -1)
📝 packages/hoppscotch-common/src/services/inspection/inspectors/__tests__/response.inspector.spec.ts (+4 -0)

📄 Description

Closes #3458

Description

As requested as a feature this PR is aimed to pass statusText from response and display it in place of the Hoppscotch Response.

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 Information

BREAKING CHANGE:
Not sure if it i breaking anything at an API level but the type defn. of HoppRESTResponse defined in packages/hoppscotch-common/src/helpers/types/HoppRESTResponse.ts has been updated to request statusText as a mandatory field.

_{🔄 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/3466 **Author:** [@kpguru20001](https://github.com/kpguru20001) **Created:** 10/21/2023 **Status:** ✅ Merged **Merged:** 12/4/2023 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `release/2023.12.0` ← **Head:** `feature/display-statusText-#3458` --- ### 📝 Commits (4) - [`4e2bf1f`](https://github.com/hoppscotch/hoppscotch/commit/4e2bf1ffb73f0effdb74ffc652bd2e8f1a994771) docs(Common): :memo: an RFC document to explain the PR and its relevant changes - [`9799b7b`](https://github.com/hoppscotch/hoppscotch/commit/9799b7bce99ccbbdd97b0bbcfeb9e9c41e49692a) feat(Common): :sparkles: pass statusText from axios res to hoppRestResponse and display - [`bca368a`](https://github.com/hoppscotch/hoppscotch/commit/bca368a9f85d95db4645eed8106be31683519d66) fix(PR-Comments): :bug: removed console logs and dead code files - [`48dd94e`](https://github.com/hoppscotch/hoppscotch/commit/48dd94e6301a36ca32a39d5d327581ae813766ca) refactor: eliminate redundancy ### 📊 Changes **5 files changed** (+25 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-common/src/components/http/ResponseMeta.vue` (+6 -1) 📝 `packages/hoppscotch-common/src/helpers/network.ts` (+1 -1) 📝 `packages/hoppscotch-common/src/helpers/types/HoppRESTResponse.ts` (+2 -1) 📝 `packages/hoppscotch-common/src/helpers/utils/statusCodes.ts` (+12 -1) 📝 `packages/hoppscotch-common/src/services/inspection/inspectors/__tests__/response.inspector.spec.ts` (+4 -0) </details> ### 📄 Description  Closes #3458 ### Description As requested as a feature this PR is aimed to pass statusText from response and display it in place of the Hoppscotch Response. <img width="395" alt="image" src="https://github.com/hoppscotch/hoppscotch/assets/25279263/26e5fbc9-e3c2-4e42-982c-37027d54cb80"> ### 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 Information BREAKING CHANGE: Not sure if it i breaking anything at an API level but the type defn. of `HoppRESTResponse` defined in `packages/hoppscotch-common/src/helpers/types/HoppRESTResponse.ts` has been updated to request statusText as a mandatory field. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>