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

Formtastic 2.0 support #527

Closed
wants to merge 5 commits into from
Closed

Conversation

ebeigarts
Copy link
Contributor

I have refactored all the input helers to use the new Formtastic 2.0 custom input convention and the code looks much cleaner now.

This is not compatible with Formtastic 1.2, since there have been a lot of changes between 1.2 and 2.0.

I can confirm that all specs and cucucmber features are passing and it is already working in one of my applications.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…inputs.
@mhuggins
Copy link

AWESOME! I'll definitely be using your fork until this is accepted by @gregbell

@rojotek
Copy link

rojotek commented Sep 24, 2011

would love to see this pulled in.

@gregbell
Copy link
Contributor

Hi @ebeigarts. Thanks for your work!

I haven't gotten a chance to go through the code yet, but I hope to get this merged in this week. We may have to support both version of formtastic for while to make it easier for users to upgrade.

@Paxa
Copy link

Paxa commented Sep 26, 2011

Thanks you bro!

@pcreux
Copy link
Contributor

pcreux commented Sep 26, 2011

@ebeigarts Thank you!

@jcuervo
Copy link

jcuervo commented Sep 26, 2011

@ebeigarts Kudos to you!

@mattvague
Copy link
Contributor

Awesome work, you deserve many cupcakes!

@macfanatic
Copy link
Contributor

Very excited about this addition!

@gregbell
Copy link
Contributor

It seems like Formtastic 2 is a drop in replacement for v1. Does anyone have any objections to setting Formtastic 2 to as dependency?

Obviously this means any application using Active Admin needs to upgrade to Formtastic 2. Anyone have any upgrade stories to share?

@mattvague - Was the upgrade seamless at VersaPay (minus Active Admin)?

@pcreux
Copy link
Contributor

pcreux commented Sep 29, 2011

Would it be hard to keep implementations for Formastic 1 & 2 (at least
until 0.4.0) otherwise? Unless it's really just a drop-in replacement. :)

φ

@ebeigarts
Copy link
Contributor Author

@pcreux it would require moving at least 2 methods from formtastic 1.2 to active admin, see #499 (comment)

@mattvague
Copy link
Contributor

@gregbell There weren't too many problems that I recall when I tried to do the upgrade aside from a couple name changes in formtastic.rb and changing a few monkey patches (and of course AA).

I'm in favour of setting 2.0 as a dependency. I'd rather force people to upgrade to 2.0 to than prevent them from upgrading to it. If worse comes to worst people can always lock to an older version of ActiveAdmin if they can't upgrade Formtastic for some reason.

@tboerger
Copy link

I'm waiting for this patch... I need formtastic 2, and i dont want to install active_admin as plugin for my new project :(

@Aze
Copy link

Aze commented Sep 30, 2011

Great job, great job!

@bdastous
Copy link

bdastous commented Oct 6, 2011

Also waiting for the FT 2.0 patch... I need the "member_label" option for my checkbox inputs :D

@mattvague
Copy link
Contributor

@gregbell @pcreux Have we come to a consensus on whether or not to merge this and drop support for < Formtastic 2.0 in future version of AA? This has been sitting here forever and I'd love to see it merged!

@mhuggins
Copy link

mhuggins commented Oct 6, 2011

Agreed!

@Zequez
Copy link
Contributor

Zequez commented Oct 10, 2011

Thanks! Hope it gets pulled in soon ^^

@radanskoric
Copy link
Contributor

Is there any news on the ETA for this? A lot of people would be grateful, including myself. :)

@szymon-przybyl
Copy link
Contributor

+1, it will also fix a bug when using active_admin and client_side_validations in the same app: #607

@jcorcuera
Copy link

Thanks! Great job! I really need this update.

@pcreux
Copy link
Contributor

pcreux commented Oct 13, 2011

Please use @ebeigarts fork until @gregbell get a chance to review & merge this pull request.

@gregbell
Copy link
Contributor

This is going to be merged in. We're going to release it as 0.4.0 and change the dependency of Active Admin to formtastic ~> 2. I should have a chance to merge it in over the next couple of days.

@gregbell
Copy link
Contributor

Thanks for this update Edgars! I just merged it in.

@gregbell gregbell closed this Oct 15, 2011
@mhuggins
Copy link

Thanks @gregbell!! :)

@jcuervo
Copy link

jcuervo commented Oct 16, 2011

Thank you very much! I think a lot of people will be more than happy with these updates!

@Zequez
Copy link
Contributor

Zequez commented Oct 17, 2011

Thanks!

@jpemberthy
Copy link
Contributor

thanks!

@tboerger
Copy link

Currently it is still not available within gemcutter :(

@jmaskell
Copy link

Just wondering if there are any plans to release with support for Formtastic 2 anytime soon? I'm not able to use other gems because active_admin is holding me back!

@mrcasals
Copy link
Contributor

@jmaskell just use the git version of Active Admin :)

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

Successfully merging this pull request may close these issues.

None yet