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

Abstract form DRY #1077

Merged
merged 2 commits into from Feb 24, 2013

Conversation

Projects
None yet
2 participants
@Ortuna
Member

Ortuna commented Feb 24, 2013

DRY some of abstract_form_builder.rb

methods field_id and field_name share alot of setup code.

@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Feb 24, 2013

Member

Awesome, thanks for all the code refactors!

Member

nesquena commented Feb 24, 2013

Awesome, thanks for all the code refactors!

nesquena added a commit that referenced this pull request Feb 24, 2013

@nesquena nesquena merged commit 16af545 into padrino:master Feb 24, 2013

1 check passed

default The Travis build passed
Details

@Ortuna Ortuna deleted the Ortuna:abstract-form-refactor branch Feb 24, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment