[PR #2239] [MERGED] feat: unsaved change popup #3686

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

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/2239
Author: @nivedin
Created: 4/6/2022
Status: Merged
Merged: 4/13/2022
Merged by: @AndrewBastin

Base: mainHead: feat/unsaved-change-popup


📝 Commits (10+)

  • 32bbd7d feat: show warning popup when changing api
  • 634b902 chore: updates
  • d415b59 refactor: migrate component to script setup on ts
  • 8d2fe3f chore: popup ui change
  • 1fc7243 refactor: migrate to setup and ts
  • dfbab0c chore: update
  • 8bf06f2 refactor: types and functional improvements
  • c75783f chore: rebase with main
  • 6d1f9fb fix: merge commits
  • 878cabc Merge branch 'main' into feat/unsaved-change-popup

📊 Changes

7 files changed (+769 additions, -240 deletions)

View changed files

📝 packages/hoppscotch-app/.gitignore (+1 -1)
📝 packages/hoppscotch-app/components/collections/my/Request.vue (+295 -117)
📝 packages/hoppscotch-app/components/collections/teams/Request.vue (+268 -100)
📝 packages/hoppscotch-app/components/history/index.vue (+134 -22)
packages/hoppscotch-app/components/http/ReqChangeConfirmModal.vue (+59 -0)
📝 packages/hoppscotch-app/helpers/types/HoppRequestSaveContext.ts (+10 -0)
📝 packages/hoppscotch-app/locales/en.json (+2 -0)

📄 Description

Closes #2207

Description

Shows a popup when there are any changes done in the current request (API, parameter, body,..) after clicking on any other collection request.

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

When a user is working on a request and made many changes and accidentally clicks on another collection request all the changes that have been done will be lost. So to prevent this problem this PR adds a function that triggers a popup when any changes are detected when the user clicks on another collection request which asks whether to save or discard the changes.

TODO

  • Team > Request
  • History

🔄 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/2239 **Author:** [@nivedin](https://github.com/nivedin) **Created:** 4/6/2022 **Status:** ✅ Merged **Merged:** 4/13/2022 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `main` ← **Head:** `feat/unsaved-change-popup` --- ### 📝 Commits (10+) - [`32bbd7d`](https://github.com/hoppscotch/hoppscotch/commit/32bbd7d4dbe850aab0afcf552d28ef260a24da3e) feat: show warning popup when changing api - [`634b902`](https://github.com/hoppscotch/hoppscotch/commit/634b9022170be1acc888c8f3c88f958922e1976b) chore: updates - [`d415b59`](https://github.com/hoppscotch/hoppscotch/commit/d415b590dd62f2b5c63e23f1dc69f9a5511299e4) refactor: migrate component to script setup on ts - [`8d2fe3f`](https://github.com/hoppscotch/hoppscotch/commit/8d2fe3fb75f97b5583ca36cfe5c088c7cba72aa7) chore: popup ui change - [`1fc7243`](https://github.com/hoppscotch/hoppscotch/commit/1fc724369587b5d83b9f9d0d2cadd303b3b35df4) refactor: migrate to setup and ts - [`dfbab0c`](https://github.com/hoppscotch/hoppscotch/commit/dfbab0c41424d75db75aef48fe4ac7efbb4a5c1e) chore: update - [`8bf06f2`](https://github.com/hoppscotch/hoppscotch/commit/8bf06f2f0a54566a307668d04e77906f9c2826e6) refactor: types and functional improvements - [`c75783f`](https://github.com/hoppscotch/hoppscotch/commit/c75783f64ef215c50c3cfdb0c1e179f56ad0ac11) chore: rebase with main - [`6d1f9fb`](https://github.com/hoppscotch/hoppscotch/commit/6d1f9fbeb830e65dfe8937f1e9d6f46e25e09252) fix: merge commits - [`878cabc`](https://github.com/hoppscotch/hoppscotch/commit/878cabcb5b7e6a55d59c8497cf5fa7de9feb0030) Merge branch 'main' into feat/unsaved-change-popup ### 📊 Changes **7 files changed** (+769 additions, -240 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-app/.gitignore` (+1 -1) 📝 `packages/hoppscotch-app/components/collections/my/Request.vue` (+295 -117) 📝 `packages/hoppscotch-app/components/collections/teams/Request.vue` (+268 -100) 📝 `packages/hoppscotch-app/components/history/index.vue` (+134 -22) ➕ `packages/hoppscotch-app/components/http/ReqChangeConfirmModal.vue` (+59 -0) 📝 `packages/hoppscotch-app/helpers/types/HoppRequestSaveContext.ts` (+10 -0) 📝 `packages/hoppscotch-app/locales/en.json` (+2 -0) </details> ### 📄 Description Closes #2207 ### Description Shows a popup when there are any changes done in the current request (API, parameter, body,..) after clicking on any other collection request. ### 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 When a user is working on a request and made many changes and accidentally clicks on another collection request all the changes that have been done will be lost. So to prevent this problem this PR adds a function that triggers a popup when any changes are detected when the user clicks on another collection request which asks whether to save or discard the changes. **TODO** - [x] Team > Request - [x] History --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 01:16:51 +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#3686
No description provided.