Skip to content

Update to fastboot@3.1.0#70

Merged
rwjblue merged 6 commits intoember-fastboot:masterfrom
rwjblue:update-fastboot
Oct 23, 2020
Merged

Update to fastboot@3.1.0#70
rwjblue merged 6 commits intoember-fastboot:masterfrom
rwjblue:update-fastboot

Conversation

@rwjblue
Copy link
Copy Markdown
Member

@rwjblue rwjblue commented Oct 16, 2020

No description provided.

@bobisjan bobisjan mentioned this pull request Oct 20, 2020
@bobisjan
Copy link
Copy Markdown
Contributor

I've tried this locally, and it looks that there is difference between fastboot@2.x and fastboot@3.x:

When I moved the result finalisation into finally block and removed this return statement, then tests passed.

@rwjblue
Copy link
Copy Markdown
Member Author

rwjblue commented Oct 23, 2020

Thank you @bobisjan!

I find this behavior to be _very odd_, but it matches what we have under
test. Specifically, it seems bizarre to call `next` and then do
_anything_ with the response still, calling `next` signifies "I'm done
here"!!?!?!!
@rwjblue rwjblue merged commit d7a504c into ember-fastboot:master Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants