Skip to content

Conversation

aferrandini
Copy link

@aferrandini aferrandini commented Feb 26, 2017

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets fixes #123
License MIT

What's in this PR?

Changes the HttpFulfilledPromise constructor parameter to mixed instead of ResponseInterface

Example Usage

return $response->then(
    function (ResponseInterface $response) use ($request) {
        if (200 <= $response->getStatusCode() && $response->getStatusCode() < 300) {
            return 'success';
        } 

        return 'fail';
    }
);

Checklist

  • Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

@aferrandini aferrandini force-pushed the hotfix/http-fulfilled-promise branch from 6e7d264 to b862c34 Compare February 26, 2017 09:08
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.

HttpFulfilledPromise constructor parameter should be mixed
1 participant