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

Grails 2.0 compatible? #1

Open
timchiu opened this issue Jun 20, 2012 · 8 comments
Open

Grails 2.0 compatible? #1

timchiu opened this issue Jun 20, 2012 · 8 comments

Comments

@timchiu
Copy link

timchiu commented Jun 20, 2012

Hi,

I really like the plugin and was wondering if there are any plans to migrate it to Grails 2.0 so that I can continue to use it with my current project?

Thanks,
Tim

@mycue1981
Copy link

Hi,

I would also dig it if the plugin would support Grails 2.0!
Any plans to improve the plugin?

Cheers
Johannes

@timchiu
Copy link
Author

timchiu commented Jul 20, 2012

+1

1 similar comment
@pjdvmalan
Copy link

+1

@johannol
Copy link

johannol commented Jun 1, 2013

Hi,

I worked a little bit on it, and i'm having now a working one on latest's Grails 2.x

The main issues i found:

  • Wrong Javascript + CSS links
  • Bug in grails about array params in render taglib (i worked arround with a custom taglib)
  • Improved the custom "getDeclaredMethods()" to filtering out the injected Grails method

Trygve Amundsen doesn't seem to maintain anymore the plugin, but i'd like to share my code for an official release with the same name, and a git push is useless if nobody care. How to proced?

I propose Trygve Amundsen to contact me at johann.ollivier-lapeyre (at Gm@ilDOTconn), and in the meantime, i'll create a grails-class-diagram2. Anyway, i'll prefer Trygve Amundsen contact me to merge everything, keep the same name and avoid confusion.

Bye.

@trygvea
Copy link
Owner

trygvea commented Jun 1, 2013

Hi,

Sorry for the long silence and inactivity on this plugin. Too much paid work, kids and real life is to 'blame'. I really appreciate the contributions and agree that the plugin should retain its name to avoid confusion.

If you create a pull request @johannol, I'll get it published asap.

Trygve

@mycue1981
Copy link

Awesome, @johannol! Thank you!

@spornitz
Copy link

spornitz commented Jan 2, 2014

Hi, folks;

This would be extremely useful to me as I am teaching fellow developers about Grails. Does the plugin now support Grails 2+?

Thanks
-Bill

@mycue1981
Copy link

@macrosak did you do some work in this area?

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

No branches or pull requests

6 participants