Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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!

[[ 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