padcom / grails-routing

Apache License 2.0
30 stars 31 forks source link

serious camel autostart bug fixed #29

Closed aahutsal closed 10 years ago

aahutsal commented 10 years ago

camel autostart bug fixed. We can't autostart camel BEFORE doWithDynamicMethods is called, coz live camel routes may call service methods which use sendMessage for example, which is NOT yet injected. Added autoStartup param value. Moved camel startup in the end of doWithDynamicMethods

padcom commented 10 years ago

Hi, I have reviewed your pull request and I feel like merging it. Stay tuned.

aahutsal commented 10 years ago

Still didn't merge? It looks like it's ready for merge.

aahutsal commented 10 years ago

Thanks for adding me to the repository. Merged recent pool request. Would be good if you release 1.2.5 version of the plugin to plugins.grails.org.

padcom commented 10 years ago

Arsen, I just added you as a collaborator to this project. I see you have made good use of it already. You can take the responsibility to release the latest version onto grails.org portal - you just need to request access to that portal.

Don't forget to add yourself to the authors list in plugin descriptor!

Good luck!

aahutsal commented 10 years ago

Thank you. I never worked with plugin release on grails.org. But I'll take a look.

2013/9/13 Matthias Hryniszak notifications@github.com

Arsen, I just added you as a collaborator to this project. I see you have made good use of it already. You can take the responsibility to release the latest version onto grails.org portal - you just need to request access to that portal.

Don't forget to add yourself to the authors list in plugin descriptor!

Good luck!

— Reply to this email directly or view it on GitHubhttps://github.com/padcom/grails-routing/pull/29#issuecomment-24381069 .

Arsen

aahutsal commented 10 years ago

Could you please help me? How could I request the access to http://grails.org/plugin/routing?

2013/9/13 Arsen A. Gutsal gutsal.arsen@softsky.com.ua

Thank you. I never worked with plugin release on grails.org. But I'll take a look.

2013/9/13 Matthias Hryniszak notifications@github.com

Arsen, I just added you as a collaborator to this project. I see you have made good use of it already. You can take the responsibility to release the latest version onto grails.org portal - you just need to request access to that portal.

Don't forget to add yourself to the authors list in plugin descriptor!

Good luck!

— Reply to this email directly or view it on GitHubhttps://github.com/padcom/grails-routing/pull/29#issuecomment-24381069 .

Arsen

Arsen

padcom commented 10 years ago

http://grails.org/Creating+Plugins

2013/9/13 Arsen A. Gutsal notifications@github.com

Could you please help me? How could I request the access to http://grails.org/plugin/routing?

2013/9/13 Arsen A. Gutsal gutsal.arsen@softsky.com.ua

Thank you. I never worked with plugin release on grails.org. But I'll take a look.

2013/9/13 Matthias Hryniszak notifications@github.com

Arsen, I just added you as a collaborator to this project. I see you have made good use of it already. You can take the responsibility to release the latest version onto grails.org portal - you just need to request access to that portal.

Don't forget to add yourself to the authors list in plugin descriptor!

Good luck!

— Reply to this email directly or view it on GitHub< https://github.com/padcom/grails-routing/pull/29#issuecomment-24381069> .

Arsen

Arsen

— Reply to this email directly or view it on GitHubhttps://github.com/padcom/grails-routing/pull/29#issuecomment-24383728 .