Y U NO ActiveModel?

[doorkeeper-gem]

Y U NO ActiveModel?

[felipeelias]

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 ?

[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.

[trungpham]

I am also interested with getting this to work with mongoid.

[nambrot]

In favor of this too (Re:Mongoid)

[trungpham]

It seems like it's possible to use Mongoid with doorkeeper by pulling epicagency@f0a5f8b
into your branch.

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!

[felipeelias]

I'll definitely take a look. Thanks

[felipeelias]

Working on this: https://github.com/applicake/doorkeeper/tree/mongoid-support

Run specs with: rake DOORKEEPER_ORM=mongoid

Completely experimental support for now

[trungpham]

very cool man. can't wait to try it out.

[l4u]

@felipeelias How do I configure it to use AR/Mongoid?
It doesn't seem to include the files inside lib/doorkeeper/models/mongoid/* ?

[l4u]

just found the answer by reading the source code :p

to use mongoid, edit config/initializers/doorkeeper.rb
add orm 'mongoid' inside the Doorkeeper.configure block

[barmstrong]

Awesome - glad to see a mongoid workaround. Thanks!

[felipeelias]

We're getting there. Still have to rebase with latest changes in the gem

[barmstrong]

Sweet, thx for the update!

[jjaffeux]

Great news !

[felipeelias]

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) 😄

[barmstrong]

Nice work! 👍

[jjaffeux]

Will try to test it this week-end

[fawzyj]

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

[felipeelias]

@fawzyj I'll update the docs today and let you know.