LaraBug / LaraBug

Laravel error reporting tool
https://www.larabug.com
MIT License
267 stars 58 forks source link

Use laravel `Http` facade for easy configuring request #85

Closed lloricode closed 1 year ago

lloricode commented 1 year ago
lloricode commented 1 year ago

I think we need to change the method names as you're using the HTTP facade instead of Guzzle. Also maybe it's better to first drop the old Laravel versions before merging this in.

What do you think @Cannonb4ll ?

Hi, already drop older laravel version, and the test is passed

Cannonb4ll commented 1 year ago

Would this be ready for review?

lloricode commented 1 year ago

Would this be ready for review?

Hi, yes, but there a conflict with branch, will fix now

lloricode commented 1 year ago

hi @Cannonb4ll all test passed and this is ready to review, thank you