Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed issue where generated collection contained request and folder in incorrect order. #736

Merged
merged 1 commit into from
Jun 12, 2023

Conversation

VShingala
Copy link
Member

Overview

While introduction of v2 interface, we introduced graphlib and use of Tree graph and preorder traversal to better organize collection folders and requests. This implementation did not add support for the folder/request creation in same order as mentioned in definition and order was reverse-alphbetical.

RCA

V2 implementation relied on certain criteria that before each request creation, the folder corresponding to it should already be created as nodes. Due to this we used to reverse sort the path and then create the Tree graph. this created the problem of incorrect order in generated collection

Fix

With this PR, we're adding support for this Tree graph creation to be in same order as mentioned in definition/ This results in also generated collection being in order as Preorder traversal of this Tree graph generates request/folder in same order as defined in definition.

@VShingala VShingala merged commit b0d83a4 into develop Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants