BCStudentSoftwareDevTeam / BCStudentSoftwareDevTeam/chemical-inventory

Expand Container to Contain Fields for Department and Person Reasponsible

Open
#138 1 comment 0 reactions 0 assignees View on GitHub
enhancement major
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/138) by Kye Hoover (Bitbucket: [eykrevooh](https://bitbucket.org/eykrevooh), GitHub: [eykrevooh](https://github.com/eykrevooh)).**

----------------------------------------

The Container needs to track information on the Department that ordered it and the Person who is Responsible.

Ex. Chemistry is Department and Dr. Saderholm is the person responsible

This requires...

1. Expanding the core container and histories data model...
* /application/models/containersModel.py
* /application/models/historiesModel.py
* Both of these should he null allowed fields
2. Expand the containerConfig to include the two new fields
* /config/containerConfig.yaml
* Fields should be optional
3. Maybe need to expand the ViewChemicalController? Possible not.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading /application/models/containersModel.py, /application/models/historiesModel.py, and /config/containerConfig.yaml to understand the existing container fields and configuration structure. Confirm whether ViewChemicalController.py is involved and inspect the relevant tests or migration workflow. Done means containers and histories can optionally track the ordering department and responsible person, with both fields represented in the configuration.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend, databases
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.