Skip to content

Add copy as curl button in profiler #158

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

Merged
merged 1 commit into from
May 4, 2017
Merged

Add copy as curl button in profiler #158

merged 1 commit into from
May 4, 2017

Conversation

fbourigault
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Related tickets fixes #108
Documentation N/A
License MIT

This add the ability to copy any request from the profiler as cURL command!

I also reworked the profiler actions. I moved all possible actions to a kind of toolbar which contains copy as cURL, toggle profile stack and toggle body buttons.

httplugbundle5

@fbourigault fbourigault added this to the Version 1.5.0 milestone May 4, 2017
@Nyholm Nyholm requested a review from sagikazarmark May 4, 2017 10:46
Copy link
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great. Good job!

@Nyholm
Copy link
Member

Nyholm commented May 4, 2017

We need php-http/message: ^1.4

@fbourigault
Copy link
Contributor Author

Should be fixed.

@Nyholm
Copy link
Member

Nyholm commented May 4, 2017

Awesome. Thanks

@Nyholm Nyholm merged commit 3b1f332 into php-http:master May 4, 2017
@fbourigault fbourigault deleted the use-curl-formatter branch May 4, 2017 12:58
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.

Use cUrl formatter
3 participants