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

Changes to support devise-neo4j #3054

Merged
merged 3 commits into from Jun 6, 2014

Conversation

cheerfulstoic
Copy link
Contributor

I've been working to get the devise-neo4j gem working with the alpha version of neo4j. I've done the work in the neo4j gem to make it support an orm_adapter, but when I ran the devise tests I hit a couple of snags which needed to be addressed in the devise gem. They are:

  • Using adapters for User and Admin in the devise tests
  • Supporting both Time and DateTime model properties

Let me know if you have any questions about these changes!

@josevalim
Copy link
Contributor

I think it would be better to explicitly call .to_adapter in the tests. Could you please amend accordingly? Thank you!

cheerfulstoic referenced this pull request in neo4jrb/activegraph Jun 6, 2014
Conflicts:
	lib/neo4j/active_node/labels.rb
@cheerfulstoic
Copy link
Contributor Author

Done! Tests ran fine locally. We'll see how Travis CI goes

josevalim pushed a commit that referenced this pull request Jun 6, 2014
Changes to support devise-neo4j
@josevalim josevalim merged commit eeb6060 into heartcombo:master Jun 6, 2014
@josevalim
Copy link
Contributor

❤️ 💚 💙 💛 💜

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

Successfully merging this pull request may close these issues.

None yet

2 participants