-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Y U NO ActiveModel? #26
Comments
I like the idea of being database agnostic. Ideally it would be nice to release this sort of feature in version 0.4/0.5. @piotrj ? |
Hmmm I don't think that just using ActiveModel would make it database agnostic. That is not what ActiveModel is for. But if we would use this and some ORM adapater (for example: https://github.com/ianwhite/orm_adapter) then yeah - that would make sense. And yeah - I think that supporting something except ActiveRecord would be awesome idea and target for 0.4/0.5. |
I am also interested with getting this to work with mongoid. |
In favor of this too (Re:Mongoid) |
It seems like it's possible to use Mongoid with doorkeeper by pulling epicagency@f0a5f8b Be warned though. You will lose SQL database support. The code needs to be refactored to work with both ActiveRecord and Mongoid. Check it out! |
I'll definitely take a look. Thanks |
Working on this: https://github.com/applicake/doorkeeper/tree/mongoid-support Run specs with: Completely experimental support for now |
very cool man. can't wait to try it out. |
@felipeelias How do I configure it to use AR/Mongoid? |
just found the answer by reading the source code :p to use mongoid, edit |
Awesome - glad to see a mongoid workaround. Thanks! |
We're getting there. Still have to rebase with latest changes in the gem |
Sweet, thx for the update! |
Great news ! |
Just rebased the branch and fix bazillion of issues with mongo. I thought it would be easier but it turned out it wasn't. This is the branch Hopefully it would be straightforward to work from mongoid. There might be issues with installation process which will try to copy migration files, so be aware (it will be fixed). If somebody who uses mongo can test this branch it would be great! Please make comments and things that you think it could be improved. Thanks for waiting (so long) 😄 |
Nice work! 👍 |
Will try to test it this week-end |
Thanks for the nice work, i was looking for it really, noe, wants to know where to get started, will be nice to have a wiki page |
@fawzyj I'll update the docs today and let you know. |
I'm talking in sake of supporting other database libraries such as Mongoid
The text was updated successfully, but these errors were encountered: