BCStudentSoftwareDevTeam / BCStudentSoftwareDevTeam/chemical-inventory

Redundant Checkout Code

Open
#117 0 comments 0 reactions 0 assignees View on GitHub
minor task
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.