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

feat: Updated react/http from ^0.8.0 to ^1.1.0 #4

Merged
merged 1 commit into from
Oct 2, 2020

Conversation

faabiopontes
Copy link
Contributor

There weren't breaking changes with the use that is made of react/http on this package.
I might be forgotting something, but all the tests from PHP Unit passed.

Please comment if improvements are necessary.

@WyriHaximus WyriHaximus added this to the 2.0.0 milestone Oct 2, 2020
@WyriHaximus WyriHaximus merged commit 6ac3895 into WyriHaximus:master Oct 2, 2020
@WyriHaximus
Copy link
Owner

There weren't breaking changes with the use that is made of react/http on this package.
I might be forgotting something, but all the tests from PHP Unit passed.

Sometimes it is that simple, thanks!

Please comment if improvements are necessary.

Not in this PR, but if you want, you can remove ringcentral/psr7 as a dependency and replace it usages with react/http's message classes instead.

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.

None yet

2 participants