[PR #2473] [MERGED] Test Results: Environment variables overflow issue fix #3831

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

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/2473
Author: @nirnejak
Created: 6/27/2022
Status: Merged
Merged: 6/27/2022
Merged by: @liyasthomas

Base: mainHead: main


📝 Commits (2)

  • 5287208 Test Results: Environment Variables overflow fix
  • 876ffb5 Test Results: Environment Variables overflow padding reduced

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 packages/hoppscotch-app/components/http/TestResultEnv.vue (+2 -2)

📄 Description

Closes #2472

Description

If we set environment variables in Test, The Test Results section shows the environment variable change,
But if the value of the variable is too long it breaks the UI, I have added a fix that will break the word in multiple lines if it's too long.

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

Screenshot of the Test Results section after changes(for large, overflowing values)

Screen Shot 2022-06-27 at 2 08 19 PM

🔄 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/2473 **Author:** [@nirnejak](https://github.com/nirnejak) **Created:** 6/27/2022 **Status:** ✅ Merged **Merged:** 6/27/2022 **Merged by:** [@liyasthomas](https://github.com/liyasthomas) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (2) - [`5287208`](https://github.com/hoppscotch/hoppscotch/commit/52872082faa96d245414e132f682bb474a5a81ae) Test Results: Environment Variables overflow fix - [`876ffb5`](https://github.com/hoppscotch/hoppscotch/commit/876ffb5292006962f647288f8410a6b9544fcc61) Test Results: Environment Variables overflow padding reduced ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-app/components/http/TestResultEnv.vue` (+2 -2) </details> ### 📄 Description <!-- Thanks for creating this pull request 🤗 Please make sure that the pull request is limited to one type (docs, feature, etc.) and keep it as small as possible. You can open multiple prs instead of opening a huge one. --> <!-- If this pull request closes an issue, please mention the issue number below --> Closes #2472 ### Description If we set environment variables in Test, The Test Results section shows the environment variable change, But if the value of the variable is too long it breaks the UI, I have added a fix that will break the word in multiple lines if it's too long. <!-- You can also choose to add a list of changes and if they have been completed or not by using the markdown to-do list syntax - [ ] Not Completed - [x] Completed --> ### Checks <!-- Make sure your pull request passes the CI checks and do check the following fields as needed - --> - [x] My pull request adheres to the code style of this project - [x] My code requires changes to the documentation - [x] I have updated the documentation as required - [x] All the tests have passed ### Additional Information Screenshot of the **Test Results** section after changes(for large, overflowing values) <img width="1044" alt="Screen Shot 2022-06-27 at 2 08 19 PM" src="https://user-images.githubusercontent.com/14347543/175900393-82a95128-c6d5-49fd-9af1-5fa6cabd141d.png"> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 01:24:47 +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#3831
No description provided.