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

Forms refactoring - Tidy up #487

Merged
merged 4 commits into from
Oct 9, 2019
Merged

Forms refactoring - Tidy up #487

merged 4 commits into from
Oct 9, 2019

Conversation

cintamani
Copy link
Contributor

From: https://eaflood.atlassian.net/browse/RUBY-693

There are some minor tidy-ups in this PR:

  • Get rid of initialize implementation on children forms that only call super
  • Make sure we always access transient_registration through its attr_accessor method
  • Move validations declaration to the top of the file, between class methods definitions and instance methods definitions

There are some minor tidy-ups in this PR:
 - Get rid of `initialize` implementation on children forms that only call `super`
 - Make sure we always access `transient_registration` through its `attr_accessor` method
 - Move validations declaration to the top of the file, between class methods definitions and instance methods definitions
@cintamani cintamani added the housekeeping Changes such as refactoring label Oct 8, 2019
@cintamani cintamani self-assigned this Oct 8, 2019
@cintamani cintamani changed the title [WIP] Forms refactoring - Tidy up Forms refactoring - Tidy up Oct 8, 2019
Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You started something in app/forms/waste_carriers_engine/bank_transfer_form.rb that I started spotting wasn't there on others, and before I knew it I'd made a rod for my own back.

I perhaps should have said remove the extra line (would have been far less instances), but I think it is better with one there 😩 😁

Other than that, I have zero issues with this!

cintamani and others added 3 commits October 9, 2019 12:17
@cintamani cintamani merged commit debec18 into master Oct 9, 2019
@cintamani cintamani deleted the tidy-up branch October 9, 2019 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Changes such as refactoring
Projects
None yet
2 participants