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

Adding support for using async pre-request scripts #145

Merged

Conversation

DivyMohan14
Copy link
Contributor

Solves #138

Hey @helloanoop as you had the other PR where you have injected more packages into the sandbox. I took a branch out of that and added my changes on top of that.

I used axios and this modified pre-request script to resolve my issue

@helloanoop
Copy link
Contributor

Nice @DivyMohan14 !
Will be testing this today and then will merge

@garv1301
Copy link

Hey @helloanoop faced a similar issue w.r.t to async scripts, by when are we planning to close this ?

@helloanoop helloanoop merged commit c36c7b4 into usebruno:main Apr 20, 2023
@helloanoop
Copy link
Contributor

@DivyMohan14 @garv1301
This is now available in the v0.12.0

@DivyMohan14 Thanks for taking time and getting async scripting to work !!

@DivyMohan14 DivyMohan14 deleted the feature/async-pre-request-scripts branch October 11, 2023 09:12
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.

3 participants