jenius-apps / nightingale-rest-api-client

A modern, resource-efficient REST API client for Windows
https://nightingale.rest
MIT License
769 stars 36 forks source link

Request imported from postman v2.1 collection lose folder structure #133

Closed roxk closed 4 years ago

roxk commented 4 years ago

Version Windows 10 2004 Nightingale 4.1.6.0

Steps to reproduce the bug

  1. Import the following postman v2.1 collection:
    {
    "info": {
        "_postman_id": "d3232d7e-a773-4953-8b80-0b5aa3fd79a5",
        "name": "Test Collection",
        "schema": "https://schema.getpostman.com/json/collection/v2.1.0/collection.json"
    },
    "item": [
        {
            "name": "Test folder",
            "item": [
                {
                    "name": "Test request",
                    "request": {
                        "method": "GET",
                        "header": [],
                        "url": {
                            "raw": "http://localhost:3001?param1=a&param2=b",
                            "protocol": "http",
                            "host": [
                                "localhost"
                            ],
                            "port": "3001",
                            "query": [
                                {
                                    "key": "param1",
                                    "value": "a"
                                },
                                {
                                    "key": "param2",
                                    "value": "b"
                                }
                            ]
                        }
                    },
                    "response": []
                }
            ],
            "protocolProfileBehavior": {}
        }
    ],
    "protocolProfileBehavior": {}
    }
  2. Observe "Test folder" is missing

Expected behavior The folder is imported.

Actual behavior The folder is not imported and Test request is at the root of the collection.

Screenshot image

dpaulino commented 4 years ago

Now that your pr was merged, I'll assign this to me so I can add the updated package to the app

dpaulino commented 4 years ago

Thanks again for your contribution @roxk. The fix should now be available in the latest update