Skip to content
This repository has been archived by the owner on Sep 4, 2019. It is now read-only.

Feathers Auth Update #131

Merged
merged 6 commits into from
Jan 11, 2017
Merged

Conversation

burn2delete
Copy link
Contributor

@burn2delete burn2delete commented Dec 24, 2016

This PR attempts to update to the new feathers-authentication-client.

It also fixes #127 which is the reason for this PR.

Seems tests are passing however the build is failing anyway?

@burn2delete
Copy link
Contributor Author

@ekryski @daffl this change is holding up ClojureScript integration for the new auth.

@daffl
Copy link
Member

daffl commented Dec 30, 2016

Thank you! I think the failing tests are because of Saucelabs being a pain, it should be fixed when merged. I'm going to restructure some of the other clients as well so this will land in a little bit for the Auk release.

@burn2delete
Copy link
Contributor Author

@daffl is there any way we can get an interm release? It's needed for a presentation for ClojureRemote at the beginning of February.

@ekryski
Copy link
Member

ekryski commented Dec 30, 2016

Should be pretty quick @flyboarder. We'll just need to check some of the docs and then generator. We're trying to wrap that stuff by end of the year.

@burn2delete
Copy link
Contributor Author

@daffl @ekryski thanks guys, I'll leave you too it then!

@daffl daffl merged commit 1951ce2 into feathersjs-ecosystem:master Jan 11, 2017
@daffl
Copy link
Member

daffl commented Jan 11, 2017

Sorry it took so long. Released as v2.0.0-pre.1 (under the pre npm tag).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants