multi-tenant / grails-multi-tenant-single-db

Multi Tenant implementation for single database environments - Use grails-filtering and grails-hibernate-hijacker
Apache License 2.0
32 stars 25 forks source link

upgrade to grails 3 #45

Closed Omasiri closed 9 years ago

Omasiri commented 9 years ago

Hello.

Please help me take a look at this upgrade. It will be good to do it on a separate branch (grails 3). I have not upgraded the scripts yet and there may be some work to be done ( it still makes use of hibernate-hijacker and friends )

scryan7371 commented 9 years ago

I noticed you did not convert over to use Spring Events but kept the HawkEventing. Did you have a look at Spring Events and decide it was not worth the effort? It would be nice to be able to drop Hawk Eventing in favor of Spring Eventing to reduce the conversion effort in the future.

Once this code has been merged I can test it on one of our 3.0 projects. Great work and thanks for taking this up.

Omasiri commented 9 years ago

Hello Scott, I did not try to drop hawk eventing because I think I don't have the deep understanding(yet) of the plugin required to make that change.

I also thought someone more knowledgeable about the plugin was already working on it.

It will be good to make the switch a move on to hibernate4

Please let mw know how we can move forward.....

-----Original Message----- From: Scott Ryan Sent: 5/13/2015 4:14 PM To: multi-tenant/grails-multi-tenant-single-db Cc: Omasiri Subject: Re: [grails-multi-tenant-single-db] upgrade to grails 3 (#45)

I noticed you did not convert over to use Spring Events but kept the HawkEventing. Did you have a look at Spring Events and decide it was not worth the effort? It would be nice to be able to drop Hawk Eventing in favor of Spring Eventing to reduce the conversion effort in the future.

Once this code has been merged I can test it on one of our 3.0 projects. Great work and thanks for taking this up.

— Reply to this email directly or view it on GitHub.