SORMAS-Foundation / SORMAS-Project

SORMAS (Surveillance, Outbreak Response Management and Analysis System) is an early warning and management system to fight the spread of infectious diseases.
https://sormas.org
GNU General Public License v3.0
292 stars 142 forks source link

Move health conditions from clinical course to the case [3] #6879

Closed kwa20 closed 2 years ago

kwa20 commented 3 years ago

Feature Description

Cases > Clinical Management > Therapy / Clinical Course / pre-existing conditions

Problem Description

Currently, the clinical management module is quite dispersed across the case entity and user roles and rights are not clearly separated for this module. E.g. pre-existing conditions, in contrast to the contact entity, are present in the clinical course of the case. This tab is only accessible for clinicians or case officers but other users should also be able to enter pre-existing conditions.

Proposed Change

Possible Alternatives

Additional Information

Frame 63

Development Specifications

bernardsilenou commented 3 years ago

@kwa20

kwa20 commented 3 years ago

@bernardsilenou Feel free to tag someone who could provide feedback, that would be quite helpful.

bernardsilenou commented 3 years ago

@Chinedar @foyiri @faatihi Please can you comment on this issue?

bernardsilenou commented 2 years ago

@faatihi @Chinedar @foyiri Please can you give your feedback on this specification?

bernardsilenou commented 2 years ago

To developer to work on this: Please remember to implement the change in the mobile app also.

bernardsilenou commented 2 years ago
bernardsilenou commented 2 years ago

@carolinverset @Candice-Louw FYI

Candice-Louw commented 2 years ago

Programming documentation to be delivered on completion of the issue (as a comment on this issue, or in the release notes):

kwa20 commented 2 years ago

I've split the initial description into 2 issues (cf. https://github.com/hzi-braunschweig/SORMAS-Project/issues/7640)

@MateStrysewske I guess this issue then has to be re-estimated.

Chinedar commented 2 years ago

Yes please, I believe we can move the pre-existing condition to the case information tab.

MateStrysewske commented 2 years ago

@bernardsilenou @kwa20 Should we change the position of the health conditions tab in the mobile app or should it stay where it is right now?

BarnaBartha commented 2 years ago
  1. Data migration from existing to new values (if applicable) Health conditions of each clinical course have been moved to the cases
  2. Changes to database structure/tables (if applicable) Cases now reference the health conditions directly, the relation between clinical course - health conditions has been removed
  3. Changes in API endpoints: The following endpoints return now with the case data also the health conditions data Example: ... "healthConditions": { "creationDate": 1643877178303, "changeDate": 1644836329236, "uuid": "T7WRAO-4AL77G-UBR5B7-L47E2NIM", "pseudonymized": false, "diabetes": "YES", "chronicLiverDisease": "YES", "malignancyChemotherapy": "YES", "chronicPulmonaryDisease": "YES", "chronicKidneyDisease": "YES", "chronicNeurologicCondition": "YES", "cardiovascularDiseaseIncludingHypertension": "YES", "immunodeficiencyIncludingHiv": "YES", "otherConditions": "not known" }, ...
    • /cases/all/{since}
    • /cases/all/{since}/{size}/{lastSynchronizedUuid}
    • /cases/query
    • /cases/query/persons
    • /cases/getduplicates
    • /cases/getduplicates/{reportDateThreshold}
    • /cases/{uuid}
JaquM-HZI commented 2 years ago

@BarnaBartha can we have the pregnancy postpartum part directly under the other pre-existing conditions please?

bernardsilenou commented 2 years ago

@MateStrysewske @BarnaBartha Sorry for responding late to https://github.com/hzi-braunschweig/SORMAS-Project/issues/6879#issuecomment-1034776411 Yes please, since the web is the reference app, it would be good to always map the mobile app design to it. Thanks

MateStrysewske commented 2 years ago

@bernardsilenou The question is where it should be moved to though. Please note that the health conditions are a separate entity and thus are not part of the case data form in the mobile app. They're an independent tab. Where should this tab be positioned?

bernardsilenou commented 2 years ago

@MateStrysewske Ohh sorry, I hope I understand now.

bernardsilenou commented 2 years ago

@BarnaBartha can we have the pregnancy postpartum part directly under the other pre-existing conditions, please?

JaquM-HZI commented 2 years ago

@bernardsilenou @BarnaBartha so maybe we just leave it as it was described in de mockup

kwa20 commented 2 years ago

tested this issue on test-de2 with commit https://github.com/hzi-braunschweig/SORMAS-Project/commits/84778017dd069508de8daa5869742c1aaf8d356f

The pre-existing conditions are currently placed underneath follow-up but should be placed above additional medical information preExistingConditionsPosition

kathsievers commented 2 years ago

ISSUE TRACKER COMMENT

kwa20 commented 2 years ago

ISSUE TRACKER COMMENT