-
Notifications
You must be signed in to change notification settings - Fork 12
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
O3-4521 : Add program tab to the visit patient scenario #77
Conversation
@jayasanka-sack @Bawanthathilan Could you review this please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, @dilankavishka! This is a valid step that we can include to the scenario.
@Bawanthathilan, I'll leave it to you to make sure we've covered all the necessary API calls. Let us know when you're happy with it!
src/test/java/org/openmrs/performance/registries/DoctorRegistry.java
Outdated
Show resolved
Hide resolved
@dilankavishka I think there's another API endpoint call for opening the program tab. Currently, you are only calling this endpoint:
but there is another endpoint being called:
|
@Bawanthathilan I update the file with the other end point. could you please take a look now? |
Thanks @dilankavishka. Please resolve the conflicts. |
src/test/java/org/openmrs/performance/http/DoctorHttpService.java
Outdated
Show resolved
Hide resolved
src/test/java/org/openmrs/performance/http/DoctorHttpService.java
Outdated
Show resolved
Hide resolved
src/test/java/org/openmrs/performance/scenarios/VisitPatientScenario.java
Outdated
Show resolved
Hide resolved
Thanks @dilankavishka 👏 |
This pull request introduces the following changes to Add Programs Tab in Visit Patient scenario:
Ticket URL: https://openmrs.atlassian.net/browse/O3-4521