SORMAS-Project icon indicating copy to clipboard operation
SORMAS-Project copied to clipboard

Enhance person edit form for TB-specific data

Open obinna-h-n opened this issue 11 months ago • 9 comments

Feature Description

After the Disease enum has been modified in (#13318) as part of epic (#13317), the person edit form needs to be updated with TB-specific variables.

Added Value/Benefit

Users will be able to add and edit TB-specific data on the person edit form in the system.

Acceptance Criteria

Entities, DTOs and Enums

  • [x] A LivingStatus enum is added with the following values
    • MIGRANT
    • FOREIGNER
    • HOMELESS
    • REGISTERED_AT_A_RESIDENCE
  • [x] The Person entity and PersonDto classes are updated with TB-specific variables
    • Country placeOfBirth
    • Country nationality
    • Date entryDate
    • LivingStatus livingStatus
    • placeOfBirth & nationality(citizenship) should be moved to Person information section under the Sex & Present condition of Person row fields.

Person edit form

  • [x] After the date of birth fields, the new TB-specific variables are displayed
    • Place of birth
    • Nationality
    • Entry date into country (If place of birth is not server country)
    • Living status
  • [x] The immunization card on the right sidebar is hidden

Implementation Details

No Response

Mockups

No Response

Additional Information

Only entry date into country is conditional on place of birth, rest of fields should be displayed when TB case is created

obinna-h-n avatar Mar 25 '25 13:03 obinna-h-n

Question for Chris when he returns:

  • What does foreigner mean in this context, if migrant is a person who moved to the country they are in, is foreigner a tourist?
  • should it be 'entry date' or 'entry date into country'? for clarity

SORMAS-EmelineJanigan avatar Apr 15 '25 08:04 SORMAS-EmelineJanigan

Tested and looks good

SORMAS-EmelineJanigan avatar Apr 15 '25 13:04 SORMAS-EmelineJanigan

@xavcol

SORMAS-EmelineJanigan avatar May 16 '25 12:05 SORMAS-EmelineJanigan

The living status should always be display, not only when a country of birth is selected.

XavCol avatar May 20 '25 10:05 XavCol

The living status should always be display, not only when a country of birth is selected.

Thanks @XavCol will fix

SORMAS-EmelineJanigan avatar May 20 '25 10:05 SORMAS-EmelineJanigan

The living status should always be display, not only when a country of birth is selected.

Thanks @XavCol will fix

Fixed

SORMAS-EmelineJanigan avatar May 20 '25 16:05 SORMAS-EmelineJanigan

The living status should always be display, not only when a country of birth is selected.

Thanks @XavCol will fix

Fixed

Fixed but moving back to testing bc not deployed yet to Test2 server

SORMAS-EmelineJanigan avatar May 22 '25 14:05 SORMAS-EmelineJanigan

@xavcol

SORMAS-EmelineJanigan avatar Jun 27 '25 08:06 SORMAS-EmelineJanigan

@SORMAS-EmelineJanigan Good for me

XavCol avatar Jun 30 '25 08:06 XavCol