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

Typo in docs comment for headersForRequest #1115

Merged
merged 1 commit into from Dec 6, 2016

Conversation

olleolleolle
Copy link
Contributor

This PR fixes a docs typo, completing the Mixin's method list.

@marcoow marcoow added this to the 1.2 milestone Dec 6, 2016
@marcoow marcoow merged commit 227fa7f into mainmatter:master Dec 6, 2016
@marcoow
Copy link
Member

marcoow commented Dec 6, 2016

Awesome, thanks! Just in time for 1.2-beta.1 ;)

@olleolleolle olleolleolle deleted the patch-1 branch December 6, 2016 14:21
@olleolleolle
Copy link
Contributor Author

olleolleolle commented Dec 6, 2016

@marcoow Oooh, I have an upcoming feature, too.

At my job, we built a DeviseTokenAuth authorizer/authenticator. And we want to offer it as a PR.

@marcoow
Copy link
Member

marcoow commented Dec 6, 2016

Cool! How is that different from the existing Devise authenticator/authorizer?

@olleolleolle
Copy link
Contributor Author

olleolleolle commented Dec 6, 2016

Oh, it's using tokens.

The docs in ember-simple-auth have a gist from 2013 to point out that Devise no longer does token-style auth.

But that gem, it uses Devise to offer token-based auth for Rails.

@olleolleolle
Copy link
Contributor Author

Sorry (I was answering the wrong question). I'll make a PR.

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.

None yet

2 participants