Skip to content

RA-1872:Add edit Privilege for Visit Dates and Visit Type on Visit Su…#383

Open
HerbertYiga wants to merge 1 commit intoopenmrs:masterfrom
HerbertYiga:RA-1872
Open

RA-1872:Add edit Privilege for Visit Dates and Visit Type on Visit Su…#383
HerbertYiga wants to merge 1 commit intoopenmrs:masterfrom
HerbertYiga:RA-1872

Conversation

@HerbertYiga
Copy link
Contributor

Ticket Id:https://issues.openmrs.org/browse/RA-1872

Description
Added edit Privilege for Visit Dates and Visit Type on Visit Summary Dashboard

@HerbertYiga
Copy link
Contributor Author

@dkayiwa how does this look to you

@dkayiwa
Copy link
Member

dkayiwa commented Mar 3, 2021

The ticket reporter needs to have the first priority.

@HerbertYiga
Copy link
Contributor Author

HerbertYiga commented Mar 3, 2021

The ticket reporter needs to have the first priority.

Ok,i pinged him on the ticket ID @dkayiwa

@HerbertYiga
Copy link
Contributor Author

HerbertYiga commented Mar 12, 2021

The ticket reporter needs to have the first priority.

@dkayiwa the reporter has tested the changes and says every thing is fine as per his last comments on the ticket

@HerbertYiga
Copy link
Contributor Author

The ticket reporter needs to have the first priority.

@dkayiwa the reporter has tested the changes and says every thing is fine as per his last comments on the ticket

added some comments here @dkayiwa

@HerbertYiga
Copy link
Contributor Author

Hi @dkayiwa does this look good for merging?

@HerbertYiga
Copy link
Contributor Author

hi dan,i think we can actually go on to merge this and close the ticket since the reporter too tested the changes and found things working as expected cc @dkayiwa

@HerbertYiga
Copy link
Contributor Author

since GitHub actions bot now closes pull requests that have not been active after some time, trying to re-ping @dkayiwa @gracepotma to see whether this is now good for merging!

<a class="right" id="editVisitDatesLink" href="#" data-visit-id="[[= id]]">${ ui.message("coreapps.task.editVisitDate.label")}</a>
[[ if (canEditVisit) { ]]
<a class="right" id="editVisitDatesLink" href="#" data-visit-id="[[= id]]">${ ui.message("coreapps.task.editVisitDate.label")}</a>
[[ if (mutipleVisitType) { ]]
Copy link
Member

@dkayiwa dkayiwa Jan 27, 2022

Choose a reason for hiding this comment

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

What is mutipleVisitType?

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.

2 participants