Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Save users from trolling #173

Merged
merged 2 commits into from
Nov 10, 2017

Conversation

cibernox
Copy link
Contributor

@cibernox cibernox force-pushed the save-users-from-trolling branch 3 times, most recently from f9d83bc to d90f493 Compare November 10, 2017 09:22
…trolling

If a user mistakenly uses `Ember.get(headers, headerName)` it will work now, but will
warn the user to use `header.get(headerName)` instead.
@cibernox cibernox force-pushed the save-users-from-trolling branch from d90f493 to b3c1064 Compare November 10, 2017 09:24
Copy link
Member

@rwjblue rwjblue left a comment

Choose a reason for hiding this comment

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

Thanks @cibernox!

Copy link
Contributor

@kratiahuja kratiahuja left a comment

Choose a reason for hiding this comment

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

LGTM

@rwjblue rwjblue merged commit 96b5058 into ember-fastboot:master Nov 10, 2017
@rwjblue
Copy link
Member

rwjblue commented Nov 10, 2017

@kratiahuja - Do you mind releasing (I can't get to it until tomorrow or Sunday)?

@kratiahuja
Copy link
Contributor

@rwjblue Sure, I'll do in an hour or two. I'll ping on this PR once the new version is out and go ahead bump other deps.

@cibernox cibernox deleted the save-users-from-trolling branch November 10, 2017 21:36
@cibernox
Copy link
Contributor Author

FWIW, this is not a pressing matter for me, no rush.

@kratiahuja
Copy link
Contributor

fastboot@1.1.1 is published.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants