BCStudentSoftwareDevTeam / BCStudentSoftwareDevTeam/chemical-inventory
Redundant Checkout Code
- Dominant language
- Python
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
**[Original report](https://bitbucket.org/laborstudents/chemical-inventory-new/issue/117) by Kye Hoover (Bitbucket: [eykrevooh](https://bitbucket.org/eykrevooh), GitHub: [eykrevooh](https://github.com/eykrevooh)).**
----------------------------------------
There are two places where you can checkout a chemical
1. CheckOutView.html snip && CheckOutController.py
2. ContainerInfo.html snip && ContainerInfoController.py
Need to consolidate these two snip into only the CheckOutView.html. Makes for less redundant code.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by comparing the checkout snippets in CheckOutView.html and ContainerInfo.html, along with CheckOutController.py and ContainerInfoController.py. Trace how each location performs checkout and identify what must remain in CheckOutView.html. Done means checkout is consolidated there and the redundant ContainerInfo.html path is removed without losing the existing behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- full-stack
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100