Skip to content

Conversation

@Daphne210
Copy link
Collaborator

@Daphne210 Daphne210 requested review from Mwanje and akileng56 and removed request for Mwanje October 17, 2024 20:22
@slubwama
Copy link
Contributor

@akileng56 this requires your extra review.

@slubwama
Copy link
Contributor

@akileng56 @jabahum help review this and get it in if it passes.

 into U4X-740-add-current-regimen-implementation-in-in-o-3-spa-ui
@akileng56
Copy link
Contributor

@Daphne210 this is supposed to edit the regimen but when u open the form, regimen field is not editable

onClick={() =>
launchPatientWorkspace('patient-form-entry-workspace', {
formInfo: {
encounterUuid: '',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a required field? if not remove it

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No it is not, it has been removed

: t('neverOnTbRegimen', 'Never on TB regimen')}
</span>
<RegimenButton
{/* <RegimenButton
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove code that is not in-use

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has been removed

@Daphne210
Copy link
Collaborator Author

@Daphne210 this is supposed to edit the regimen but when u open the form, regimen field is not editable

The clinical implementation of changing a regimen is only through regimen switch/substitution but in plain English it's editing

@Daphne210
Copy link
Collaborator Author

Screenshot (843)
@Daphne210 this is supposed to edit the regimen but when u open the form, regimen field is not editable

Have changed the word to "Change Regimen" don't know if that's better or worse

@akileng56 akileng56 merged commit ea07ed4 into METS-Programme:master Oct 31, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants