Skip to content

Move tool checkout validations into the model#361

Open
tomas-goncalves wants to merge 1 commit intosc0v:masterfrom
tomas-goncalves:move-tool-checkout-validation-to-model
Open

Move tool checkout validations into the model#361
tomas-goncalves wants to merge 1 commit intosc0v:masterfrom
tomas-goncalves:move-tool-checkout-validation-to-model

Conversation

@tomas-goncalves
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread app/models/checkout.rb Outdated
Comment thread app/controllers/applets/ppe_collection_controller.rb
Comment thread app/controllers/applets/ppe_collection_controller.rb
@merichar
Copy link
Copy Markdown
Member

Please ensure all checkout actions are protected with the appropriate can? authorization check.

Comment thread app/controllers/applets/ppe_distribution_controller.rb
Comment thread app/controllers/tools/checkouts_controller.rb Outdated
Comment thread app/models/checkout.rb Outdated
- Add participant_signed_waiver, participant_belongs_to_org, and
  not_already_checked_in validations to Checkout model
- Add checkin instance method
- Add checkout_batch class method (map/partition style matching ScissorLiftCheckout)
- Update Tools::CheckoutsController to use checkout_batch and checkin
- Update Applets::PPECollectionController to use checkin
- Update Applets::PPEDistributionController to use find_by and fix double create+save
@tomas-goncalves tomas-goncalves force-pushed the move-tool-checkout-validation-to-model branch from 39761b2 to 01904ab Compare March 26, 2026 06:39
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