-
Notifications
You must be signed in to change notification settings - Fork 33
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
Array Support For Loading Variables From Postman #9
base: main
Are you sure you want to change the base?
Conversation
README.md
Outdated
@@ -224,18 +224,23 @@ The extension must be added to a parameter for consumption. This could be a head | |||
```yaml | |||
parameters: | |||
bookId: | |||
name: bookId | |||
name: accountId |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why change the variable name?
I don't mind, but you didn't change it on line 226. I'm just curious why you're changing it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I missed 226 😅
As for why I changed it to an account scenario instead of a book, at the time I thought it made more sense. In retrospect, book works just as well. I'll make a few changes and push it up. Would you mind adding me as a contributor for my previous work?
im struggling to see the use case for this. I would much rather like to see array support in save 🤷♂️ for ex if I call a list or create endpoint I could get the id of the first object (if response returns multiple, or array indexing on a sub property) and store it in a variable |
Added support for arrays on the loading end of using postman variables for my own purposes and decided to share it here in case it would be a welcomed addition.