starred/yaak
1
0
Fork 0
mirror of https://github.com/mountain-loop/yaak.git synced 2026-01-13 20:53:30 +01:00
Code Issues 8 Packages Projects Releases 10 Wiki Activity

[PR #110] [MERGED] Improved querystring import on paste #46

New Issue
Closed
opened 2025-12-29 07:19:03 +01:00 by adam · 0 comments
adam commented 2025-12-29 07:19:03 +01:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/110
Author: @gschier
Created: 9/25/2024
Status: ✅ Merged
Merged: 9/25/2024
Merged by: @gschier

Base: master ← Head: better-query-import

📝 Commits (4)

  • fb9bc78 Query import on paste replaces entire URL
  • d95a597 Remove unnecessary compare
  • 4a2f6a3 Tweak
  • a101d5e Only overwrite if there's a querystring

📊 Changes

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

View changed files

📝 src-web/components/RequestPane.tsx (+2 -1)
📝 src-web/components/UrlBar.tsx (+3 -0)
📝 src-web/components/core/Editor/Editor.tsx (+17 -2)
📝 src-web/components/core/Input.tsx (+3 -0)
📝 src-web/hooks/useImportQuerystring.ts (+11 -22)

📄 Description

This PR:

  • Only imports querystring params if entire URL is selected (overwrite)
  • Don't import querystring if URL doesn't contain one
  • Removes all existing params before populating new ones

Fixes #105

🔄 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/mountain-loop/yaak/pull/110 **Author:** [@gschier](https://github.com/gschier) **Created:** 9/25/2024 **Status:** ✅ Merged **Merged:** 9/25/2024 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `master` ← **Head:** `better-query-import` --- ### 📝 Commits (4) - [`fb9bc78`](https://github.com/mountain-loop/yaak/commit/fb9bc7814ea2e900d91ceea0c7d41ea647ab3cf2) Query import on paste replaces entire URL - [`d95a597`](https://github.com/mountain-loop/yaak/commit/d95a5978bfb2d836f53116dc23221aa6be162c91) Remove unnecessary compare - [`4a2f6a3`](https://github.com/mountain-loop/yaak/commit/4a2f6a377a2aa7cf6fc05fb85bb1a87f1a0a8438) Tweak - [`a101d5e`](https://github.com/mountain-loop/yaak/commit/a101d5ec1d97b8704d04c65ee9f5a2629692b1e3) Only overwrite if there's a querystring ### 📊 Changes **5 files changed** (+36 additions, -25 deletions) <details> <summary>View changed files</summary> 📝 `src-web/components/RequestPane.tsx` (+2 -1) 📝 `src-web/components/UrlBar.tsx` (+3 -0) 📝 `src-web/components/core/Editor/Editor.tsx` (+17 -2) 📝 `src-web/components/core/Input.tsx` (+3 -0) 📝 `src-web/hooks/useImportQuerystring.ts` (+11 -22) </details> ### 📄 Description This PR: - Only imports querystring params if entire URL is selected (overwrite) - Don't import querystring if URL doesn't contain one - Removes all existing params before populating new ones Fixes #105 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 07:19:03 +01:00
adam closed this issue 2025-12-29 07:19:03 +01:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/yaak#46
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?