profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/SwathiVarkala/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

Bahmni/bahmni-core 81

Core OpenMRS modules for Bahmni (including ERP & ELIS Atom Feed Clients)

Bahmni/bahmni-vagrant 9

Management of Vagrant box using Packer. Out-Of-The-Box bahmni on CentOS 6.7

Bahmni/bahmni-offline 5

Repo to hold bahmni offline code. This will support Chromium app and Android app as of now.

Bahmni/emr-functional-tests 4

emr functional tests

akhil-ghatiki/foreCastAlexSkillBackend 3

This is a custom backend logic for Alexa skill that forecasts the retail demand

Bahmni/bahmni-docker 3

Configuration required to build and run bahmni docker images

Bahmni/bahmni-java-utils 3

This repository has various functional utilities needed by bahmni.

Bahmni/bahmni-connect 2

Front end for the Bahmni Connect, written in AngularJS

Bahmni/bahmni-docker-old 2

DEPRECATED - This repository contains the relevant docker configuration for setting up Bahmni

Bahmni/bahmni-gauge 2

Bahmni Automation Test Suite using Gauge Framework

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-958:Errors from the backend should marked the field that caused the error

@jwnasambu the issue of pull rate limit?

jwnasambu

comment created time in 6 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-958:Errors from the backend should marked the field that caused the error

@dkayiwa Am still fixing the issue but I will gladly communicate once I have pushed the changes. Thanks for asking.

jwnasambu

comment created time in 6 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-958:Errors from the backend should marked the field that caused the error

@jwnasambu is the pull rate limit error still a problem?

jwnasambu

comment created time in 6 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-958:Errors from the backend should marked the field that caused the error

@dkayiwa thanks for the help. @hadijahkyampeire the ticket is not complete that is why its in draft state

jwnasambu

comment created time in 8 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-958:Errors from the backend should marked the field that caused the error

@dkayiwa, @ibacher Kindly how do I solve this persistent error `ERROR: Service 'solr' failed to build: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

The command "bash ./start_local_instance.sh" exited with 1. ` causing the travis failure?

jwnasambu

comment created time in 8 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-965:Set Source to whatever user want it to be (maybe “Your Recent Sources” instead of hardcoded)

@hadijahkyampeire thanks for the clarification. Kindly, it sounds good when the tickets have clear description of what is expected if possible with diagram(s).

jwnasambu

comment created time in 8 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-978:Update SignUp links to new OCL links

Coverage Status

Coverage remained the same at 44.699% when pulling 9144dc11c3b6fe25b2cb25b64ae8631f5250cfc4 on suruchee:OCLOMRS-978 into da8870f23828963161d9f26a05d0acc5b27a73bd on openmrs:master.

suruchee

comment created time in 13 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-978:Update SignUp links to new OCL links

@hadijahkyampeire thanks for your review, I actually updated it as it was before. please review the updated link.

suruchee

comment created time in 14 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-958:Errors from the backend should marked the field that caused the error

@jwnasambu why did you change this PR to a draft? what's up with it?

jwnasambu

comment created time in 14 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-932:General Filter: Hide Retired Concepts & Add Filter option to Include Retired Concepts

@jwnasambu please rebase your branch with master to get the test fixes.

jwnasambu

comment created time in 14 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-932:General Filter: Hide Retired Concepts & Add Filter option to Include Retired Concepts

@ibacher please review this, I think we need to maintain the 1 and 0, if anything we actually need to change the true and false since APIv2 is using 1 and 0, we need to spec this ticket well because it looks like it has been confusing over time.

jwnasambu

comment created time in 14 hours

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-978:Update SignUp links to new OCL links

Also, rebase with master to get the fix for those tests.

suruchee

comment created time in 14 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-115:Un Ignore SearchActiveVisitTest

could you please share those changes you thing are needed to we added, otherwise it works as expected on headless =true

sherrif10

comment created time in 15 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-115:Un Ignore SearchActiveVisitTest

There is no more work to be done here i guess

@sherrif10 you may be right. While testing your changes I realised there could be some little work to be done. If you do not mind test these changes once again with chromeheadless set to true. Some thing like headless=true

sherrif10

comment created time in 15 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-115:Un Ignore SearchActiveVisitTest

There is no more work to be done here i guess, i tested this work on both chrome driver and firefox driver and all was well unless am getting your point mistakenly thanks

sherrif10

comment created time in 15 hours

push eventopenmrs/openmrs-ocl-client

nk183

commit sha 6f2b6b694214877c6a763855a91412c7f33cbe02

OCLOMRS-979 : When creating a dictionary, the selected owner is now visible

view details

kyampeire Hadijah

commit sha da8870f23828963161d9f26a05d0acc5b27a73bd

Merge pull request #692 from nk183/OCLOMRS-979 OCLOMRS-979 : When creating a dictionary, the selected owner is now v…

view details

push time in 15 hours

PR merged openmrs/openmrs-ocl-client

OCLOMRS-979 : When creating a dictionary, the selected owner is now v…

…isible

JIRA TICKET NAME:

OCLOMRS-979

Summary:

When creating a dictionary, the selected owner was not visible after changes image

+0 -1

1 comment

1 changed file

nk183

pr closed time in 15 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-115:Un Ignore SearchActiveVisitTest

Have tested the changes locally but there could be some more work to do. Do you mind testing these changes with chromeheadless instance

sherrif10

comment created time in 15 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-115:Un Ignore SearchActiveVisitTest

Hello @kaweesi could we merge this please

sherrif10

comment created time in 15 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74:Failing Test:Ignored AddFormTests

     private static String REST_PATH_2 = "'\"]";     private static By CALENDAR_EYE_FORM = By.id("w1-display");     private static By DATAPICKER_YEAR = By.className("ui-datepicker-year");-    private static By DATAPICKER_MONTH = By.className("ui-datepicker-moth");+    private static By DATAPICKER_MONTH = By.className("ui-datepicker-month");

Pr to fix the error logged in here https://github.com/openmrs/openmrs-distro-referenceapplication/pull/359

sherrif10

comment created time in 15 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74:Failing Test:Ignored AddFormTests

     private static String REST_PATH_2 = "'\"]";     private static By CALENDAR_EYE_FORM = By.id("w1-display");     private static By DATAPICKER_YEAR = By.className("ui-datepicker-year");-    private static By DATAPICKER_MONTH = By.className("ui-datepicker-moth");+    private static By DATAPICKER_MONTH = By.className("ui-datepicker-month");

sure this is a typo error, Last meeting on TAC cal we discussed of resolving these bugs along uitest automatation Test

sherrif10

comment created time in 17 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74:Failing Test:Ignored AddFormTests

     private static String REST_PATH_2 = "'\"]";     private static By CALENDAR_EYE_FORM = By.id("w1-display");     private static By DATAPICKER_YEAR = By.className("ui-datepicker-year");-    private static By DATAPICKER_MONTH = By.className("ui-datepicker-moth");+    private static By DATAPICKER_MONTH = By.className("ui-datepicker-month");

are you sure this class is renamed in the hosting module? if not you can have this refactoring/renaming change in a separate ticket such that it's all tracked at once

sherrif10

comment created time in 18 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74:Failing Test:Ignored AddFormTests

+package org.openmrs.reference.page;++import org.openmrs.uitestframework.page.Page;+import org.openqa.selenium.By;++public class HtmlFormsPage extends Page {++    private static final By NEW_HTML_FORM = By.cssSelector("a[href='htmlForm.form']");+    private static final By NAME = By.name("form.name");+    private static  By DESCRIPTION = By.name("form.description");+    private static  By SAVE_FORM = By.cssSelector("input[type=\"submit\"]");+    private  static By VERSION = By.name("form.version");++    public HtmlFormsPage(ManageHtmlFormsPage page) {+        super(page);+    }++   public void setNAME(String name) throws InterruptedException {+        driver.findElement(NAME).clear();+        driver.findElement(NAME).sendKeys("newFormTest");+        driver.findElement(NAME).getAttribute("value");+    }+    public  void setDESCRIPTION(String description) {+       driver.findElement(DESCRIPTION).clear();+      driver.findElement(DESCRIPTION).sendKeys("description");+      driver.findElement(DESCRIPTION).getAttribute("value");+    }+   public void setVERSION(String integer){

these attribute setting methods can all be merged into one method such as createNewTestForm()

sherrif10

comment created time in 18 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74:Failing Test:Ignored AddFormTests

  */ public class AddFormTest extends TestBase {     private HomePage homePage;+    private static final By NEW_HTML_FORM = By.cssSelector("a[href='htmlForm.form']");+    private static final By NAME = By.name("form.name");+    private static  By DESCRIPTION = By.name("form.description");+    private  static By VERSION = By.name("form.version");+    private static  By SAVE_FORM = By.cssSelector("input[type=\"submit\"]");+    private static  By HOME = By.id("homeNavLink");+     private HeaderPage headerPage;     private ManageFormsPage manageForm;     private ClinicianFacingPatientDashboardPage patientDashboardPage;+    private AdministrationPage administrationPage;+    private ManageHtmlFormsPage manageHtmlFormsPage;+    private HtmlFormsPage htmlFormsPage;      @Before     public void setUp() throws Exception {         homePage = new HomePage(page);-        assertPage(homePage);+        homePage.goToAdministration().clickOnManageHtmlForms().clickOnNewHtmlForm();+        driver.manage().timeouts().implicitlyWait(0,TimeUnit.SECONDS);++        driver.findElement(NAME).clear();

you can move all this page functionality into the respective page @sherrif10

sherrif10

comment created time in 18 hours

pull request commentopenmrs/openmrs-module-coreapps

RA-1711: Program Enrollment Widget should allow enrollment date of program to be equal to completion date of previous program

If they do not exist already, adding some tests for the comparator would be great. 😊

mogoodrich

comment created time in 18 hours

issue openedBahmni/openmrs-module-bahmniapps

ADT patient page launched from patient has no option to select bed after clicking twice on dropdown bar

Hi Developer Team,

i found following Bug:

When i go to the ADT page from the patient and try to e.g. admit the patient, the beds disappear when i open and close the dropdown.

ADTBug

If i want to look through my departments as user, i cannot go back because the beds are not displayed anymore.

On the InPatient Module, there is already a green button on the right side of the header which allows switching.

Bildschirmfoto 2021-04-17 um 15 42 12

I think just enabling this button in this mentioned page before solves this issue.

Current Workaround is a page reload. (But i think this is not that user-friendly also on tablets)

I would like to fix it and open PR since it should be a simple AngularJS condition somewhere, if you accept this issue.

Best regards, Johannes

created time in 2 days

pull request commentopenmrs/openmrs-ocl-client

OCLOMRS-979 : When creating a dictionary, the selected owner is now v…

Coverage Status

Coverage remained the same at 44.699% when pulling 6f2b6b694214877c6a763855a91412c7f33cbe02 on nk183:OCLOMRS-979 into 4f22f67d73c32cd04b454b8bf33b91c69855e8e6 on openmrs:master.

nk183

comment created time in 2 days

PR opened openmrs/openmrs-ocl-client

OCLOMRS-979 : When creating a dictionary, the selected owner is now v…

…isible

JIRA TICKET NAME:

OCLOMRS-979

Summary:

When creating a dictionary, the selected owner was not visible image

+0 -1

0 comment

1 changed file

pr created time in 2 days

Pull request review commentopenmrs/openmrs-module-htmlformentry

HTML-419:EncounterProviderAndRole and encounterProvider tags should display retired providers at the bottom,also add includeRetired

 public String generateHtml(FormEntryContext context) { 		sb.append("</option>"); 		 		for (Provider provider : providers) {-			sb.append("\n<option ");-			if (initialValue != null && initialValue.equals(provider))-				sb.append("selected=\"true\" ");-			sb.append("value=\"" + provider.getId() + "\">")-			        .append(provider.getPerson() != null-			                ? HtmlFormEntryUtil.getFullNameWithFamilyNameFirst(provider.getPerson().getPersonName())-			                : provider.getName())-			        .append("</option>");+			if (provider.getRetired()) {

Would be nice to have a screenshot of how this looks.

HerbertYiga

comment created time in 2 days