github-starred/yaak
2
0
Fork 0
mirror of https://github.com/mountain-loop/yaak.git synced 2026-08-26 22:03:28 -05:00
Code Issues 124 Packages Projects Releases 97 Wiki Activity

[PR #490] [MERGED] Support string-based url.path in Postman importer. #3339

New Issue
Closed
opened 2026-07-15 02:02:38 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-07-15 02:02:38 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/490
Author: @JvE-iO
Created: 7/1/2026
Status: ✅ Merged
Merged: 7/1/2026
Merged by: @gschier

Base: main ← Head: bugfix/allow-string-path-in-postman-import

📝 Commits (1)

  • 025b2a1 Support string-based url.path in Postman importer.

📊 Changes

2 files changed (+36 additions, -2 deletions)

View changed files

📝 plugins/importer-postman/src/index.ts (+6 -2)
📝 plugins/importer-postman/tests/index.test.ts (+30 -0)

📄 Description

Summary

Old postman collections have the url path as strings like api/v3/users, while newer collections have arrays like ['api','v3','users']. The yaak importer-postman plugin silently dropped the string version, making all paths empty.

This fix explicitly handles both types.

Submission

  • This PR is a bug fix.
  • If this PR is not a bug fix, I linked the feedback item where @gschier explicitly gave me permission to work on it.
  • I have read and followed CONTRIBUTING.md.
  • I tested this change locally.
  • I added or updated tests when reasonable.
  • I added screenshots or recordings for UI changes when reasonable.

🔄 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/490 **Author:** [@JvE-iO](https://github.com/JvE-iO) **Created:** 7/1/2026 **Status:** ✅ Merged **Merged:** 7/1/2026 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `main` ← **Head:** `bugfix/allow-string-path-in-postman-import` --- ### 📝 Commits (1) - [`025b2a1`](https://github.com/mountain-loop/yaak/commit/025b2a15be6b76fa97df7a9eb9cdd38b26ebaefc) Support string-based `url.path` in Postman importer. ### 📊 Changes **2 files changed** (+36 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `plugins/importer-postman/src/index.ts` (+6 -2) 📝 `plugins/importer-postman/tests/index.test.ts` (+30 -0) </details> ### 📄 Description ## Summary Old postman collections have the url path as strings like `api/v3/users`, while newer collections have arrays like `['api','v3','users']`. The yaak importer-postman plugin silently dropped the string version, making all paths empty. This fix explicitly handles both types. ## Submission - [x] This PR is a bug fix. - [ ] If this PR is not a bug fix, I linked the feedback item where @gschier explicitly gave me permission to work on it. - [x] I have read and followed [`CONTRIBUTING.md`](CONTRIBUTING.md). - [x] I tested this change locally. - [x] I added or updated tests when reasonable. - [x] I added screenshots or recordings for UI changes when reasonable. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-07-15 02:02:38 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No labels pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
GiteaMirror ninjasurge (Kaehlen Rog)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/yaak#3339