Reference Application 2.9 Testing

release-testing
Tags: #<Tag:0x00007f23d985be58>

(Reuben Varghese) #21

Hi @c.antwi , the testing phase has already begun. In fact, I was hoping to proceed towards final release should we not have any bugs. So far there have been no bug reports. Before the final release however, I’ll have to create a small commit to include the attachments module


(Daniel Kayiwa) #22

@reubenv the fact that you want to include another module means that you will have to update the testing server and then call for another round of testing, before you can release.

Do you have a notes pad where people type the results of their testing and any bugs or findings?

Here is an example for the last release: RefApp 2.8.0 volunteers needed


(Reuben Varghese) #23

Hi @dkayiwa, I have raised a pr for adding the Attachments module to the Ref App distro here. I was unaware of the notepad and naively thought that the bugs would be reported in the thread :sweat_smile: . I have also updated the op to include the same.

Notepad


(Daniel Kayiwa) #24

@reubenv did you run your changes and confirm that you are actually able to add attachments to a patient from the patient dashboard?


(Dimitri R) #25

@dkayiwa @reubenv and others, am I correct to assume that the reason why Ref App and Core Apps have not been released yet is precisely because the current QA process is still ongoing?

Cc @amine


(Cynthia Antwi) #26

That is correct.


(Reuben Varghese) #27

Yes, @dkayiwa, I have commented on the task and also mentioned on IRC but I guess you were offline.

To reiterate, yes it is working :smile:


(Reuben Varghese) #28

Yes @mksd, that’s correct. We will probably have the release in the next few days once the testing phase ends(should there be no major issues) :slight_smile:


(Daniel Kayiwa) #29

@mksd did you say that coreapps was not released? I thought we released it a number of weeks ago.


(Daniel Kayiwa) #30

@reubenv are you sure you added them without any errors and you are able to view them on the patient dashboard, on a brand new setup? Are you also monitoring the errors reported on the notepad?


(Dimitri R) #31

The distro still packages Core Apps 1.21.0-SNAPSHOT (when master = 2cf0d92):

<coreappsVersion>1.21.0-SNAPSHOT</coreappsVersion>

As for Core Apps its latest commit right now contains a POM that is still versioned at 1.21.0-SNAPSHOT (when master = da52d19):

<artifactId>coreapps</artifactId>
<version>1.21.0-SNAPSHOT</version>

(Daniel Kayiwa) #32

The agreed version on the wiki was 1.20.0 but since then, a commit has been made for AM-184 which is good enough to be considered for the reference application 2.9.0 release.


(Dimitri R) #33

Ok so Core Apps is virtually released and packaged as 1.21.0 and the Ref App module is the last one undergoing QA right?


(Daniel Kayiwa) #34

Depending on the test results, we may need to make new releases for a couple of modules to incorporate the subsequent fixes. Core Apps is just one of them.


(Reuben Varghese) #35

@dkayiwa , I just looked at the issue reported in the notes. That particular concept was what we had removed as part of updating the concepts to the latest version of CIEL. Here’s the commit for the same.

So, in my opinion, we could fix this in two ways: a) We could point the sticky notes value to another concept ( if that’s okay, not really sure) Or b) We could add the concept back

What course of action do you feel would be suitable?


(Daniel Kayiwa) #36

Can you ask @akanter if that concept is already part of the latest CIEL dictionary?


(Andrew Kanter) #37

165095 - General patient note is still in CIEL.


(Daniel Kayiwa) #38

@reubenv did you use the latest of the CIEL dictionary?


(Andrew Kanter) #39

It has been in there since April 2018.


(Reuben Varghese) #40

@dkayiwa , in that case I guess what must’ve happened is that when we restored the mdsbuillder server, the latest version of CIEL wasn’t loaded. As per this discussion, I inferred that it is the latest. Since it doesn’t seem like it, we’ll have to go over that part again I suppose

Also, thanks @akanter