human-essentials icon indicating copy to clipboard operation
human-essentials copied to clipboard

Add missing kit rspecs, DRY up kit base items and report service

Open jimmyli97 opened this issue 1 year ago • 1 comments

Split off from PR #4585 Various refactors and documentation changes

Description

  • Rename item.kits scope to item.housing_a_kit
  • DRYs up kit base_item seeding
  • Removes duplicated code and rspec from ChildrenReportService
  • Add rspecs to:
    • prevent kit base_item deletion
    • test item.is_in_kit?, item.loose, item.housing_a_kit
  • Add documentation

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Documentation update

How Has This Been Tested?

passes test suite

jimmyli97 avatar Sep 24 '24 02:09 jimmyli97

@jimmyli97 FYI: We had an urgent fix that required all the senior contributors this week , so we didn't get to look at this again. Hopefully this week will go better.

cielf avatar Sep 28 '24 17:09 cielf

@jimmyli97 are you able to finish this off?

dorner avatar Nov 10 '24 15:11 dorner

Hey @jimmyli97 -- Just checking if you are still working on this. If we don't hear from you in a couple weeks, we'll assume not. Thank you for all your work on this in either case!

cielf avatar Nov 10 '24 15:11 cielf

If someone wants to finish this that's fine, otherwise I can hopefully get to this next week

jimmyli97 avatar Nov 10 '24 17:11 jimmyli97

All good on my side. @cielf did you want to kick the tires?

dorner avatar Nov 27 '24 01:11 dorner

Yes. Should be able to fit it in today or tomorrow.

cielf avatar Nov 27 '24 13:11 cielf

Looks good after a light kicking.

cielf avatar Nov 28 '24 14:11 cielf

@jimmyli97: Your PR Add missing kit rspecs, DRY up kit base items and report service is part of today's Human Essentials production release: 2024.12.15. Thank you very much for your contribution!

github-actions[bot] avatar Dec 15 '24 16:12 github-actions[bot]