Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

New properties on HealthRisk #332

Closed
karolikl opened this issue Dec 14, 2017 · 5 comments
Closed

New properties on HealthRisk #332

karolikl opened this issue Dec 14, 2017 · 5 comments
Assignees
Labels
Milestone

Comments

@karolikl
Copy link
Collaborator

karolikl commented Dec 14, 2017

On a HealthRisk, the ConfirmedCase, SuspectedCase and ProbableCase properties should be removed, and a new should be added: CaseDefinition (string).

KeyAction (string) should also be added to the HealthRisk.

The CaseDefinition of a health risk should be editable on a global level and must therefore be included in HealthRiskController. There is no POST endpoint here at the moment, this needs to be created as part of this issue.

The CaseDefinition should also be editable on a project level and must therefore be included in ProjectsHealthRiskController.SetHealthRisks(...).

@karolikl karolikl changed the title New property on HealthRisk: CaseDefinition New properties on HealthRisk Jan 5, 2018
@karolikl karolikl self-assigned this Jan 16, 2018
karolikl added a commit to karolikl/cbs that referenced this issue Jan 17, 2018
karolikl added a commit to karolikl/cbs that referenced this issue Jan 24, 2018
@Velcrow81
Copy link
Collaborator

Event under Surveillance (might need to find a better name since event is not very descriptive) should also be added. It says something about who or what the health risk relates to. Allowed values are Human, Animal, Environment, Pregnant woman. Unusual event covers all, so we might need a value called All? Or a list of things? What do you think @karolikl and @aninek ?

@karolikl karolikl removed their assignment Feb 25, 2018
@karolikl
Copy link
Collaborator Author

Since my two commits (to my fork, not this repo), there have been massive changes to the Admin project. I'm therefore deassigning myself from the issue in case anyone wants to grab it before I have to time look into the merge conflicts.

@AG-83 AG-83 self-assigned this Mar 6, 2018
@AG-83
Copy link
Collaborator

AG-83 commented Mar 6, 2018

Not done: The CaseDefinition should also be editable on a project level and must therefore be included in ProjectsHealthRiskController.SetHealthRisks(...).

AG-83 pushed a commit to cbsadmin/cbs that referenced this issue Mar 6, 2018
… CaseDefinition. Post for add and edit HealthRisk, delete functionality is added.
@dageirikboe dageirikboe added this to the MVP milestone Apr 5, 2018
@dageirikboe dageirikboe added the Admin Admin Bounded context label Apr 5, 2018
@eprom
Copy link
Collaborator

eprom commented Apr 6, 2018

@AG-83 would you like to stay on this issue? Seems you have resolved this issue, could you make a PR?

@AG-83
Copy link
Collaborator

AG-83 commented Apr 7, 2018 via email

AG-83 pushed a commit to AG-83/cbs that referenced this issue Apr 9, 2018
karolikl added a commit that referenced this issue Apr 11, 2018
#332 New properties on HealthRisk, #344 Edit and save a HealthRisk
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants