postmanlabs / postman-app-support

Postman is an API platform for building and using APIs. Postman simplifies each step of the API lifecycle and streamlines collaboration so you can create better APIs—faster.
https://www.postman.com
5.78k stars 837 forks source link

PR folder structure elucidation #8109

Closed spencerrichardhenry closed 3 years ago

spencerrichardhenry commented 4 years ago

The new pull request feature is a wonderful improvement on version control, but it leaves our team wanting some better elucidation on folder structure changes. This is particularly important because the ordering and placement of tests holds functional meaning, since there are folder level tests as well as because the tests run based on folder order.

When you hover over the exclamation by a [MOVED] tag (see image below), it tells you what folder the request has been moved to. We would highly prefer for this to show the entire file path of where it came from and where it went to. see below for an idea of what this would look like. image image

Additionally, it would be nice to have some sort of review of the folder structure in the PR view (maybe in overview). Something like this: image where hovering over the yellow text would tell you where it moved from. This could replace the current implementation of showing a folder [MODIFIED] and then shwoing the Folders Order, possibly by just showing a copy of the folder structure before and a copy after side by side.

The implementation is less important than just having some way to understand folder structure changes better within a pull request.

ankit-m commented 3 years ago

You can now see the folder structure in PR diff view.