Skip to content

Moved Scissor Lift Checkout Validation logic to the model#358

Merged
merichar merged 4 commits intosc0v:masterfrom
tomas-goncalves:move-scissor-lift-business-logic
Feb 20, 2026
Merged

Moved Scissor Lift Checkout Validation logic to the model#358
merichar merged 4 commits intosc0v:masterfrom
tomas-goncalves:move-scissor-lift-business-logic

Conversation

@tomas-goncalves
Copy link
Copy Markdown
Contributor

No description provided.

@tomas-goncalves tomas-goncalves changed the title Moved Scissor Lift Validation logic to functions on the model Moved Scissor Lift Validation logic to the model Dec 27, 2025
@tomas-goncalves tomas-goncalves changed the title Moved Scissor Lift Validation logic to the model Moved Scissor Lift Checkout Validation logic to the model Dec 27, 2025
Comment thread app/models/scissor_lift_checkout.rb Outdated
Comment thread app/models/scissor_lift_checkout.rb Outdated
Comment thread app/models/scissor_lift_checkout.rb Outdated
Copy link
Copy Markdown
Member

@merichar merichar left a comment

Choose a reason for hiding this comment

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

Good work moving validations into the model. A few changes needed before this is ready:

  • checkin and renew_for should use save! instead of save
  • Please remove checkout_batch

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants