EyeSeeTea / SurveillanceCambodiaApp

Mobile application designed to report cases of malaria (to a DHIS2 server) for Cambodia (pictureapp blessed repository)
GNU General Public License v3.0
0 stars 0 forks source link

qispsk@gmail.com on build #25: មិនស្គាល់ or Unrecognized was shown here as I have already… #56

Closed adrianq closed 8 years ago

adrianq commented 8 years ago

Feedback from qispsk@gmail.com : មិនស្គាល់ or Unrecognized was shown here as I have already raised in previous build S4.1 build 24

Build25
Device typeLG-H324
Screen size480
Screen size480px by 786px
Battery57% Unplugged
Memory free124 MB / 711 MB

Link to buddybuild feedback from build 25

ifoche commented 8 years ago

Copying here from my email: We're trying to reproduce this bug without any success. Remember that there are still a bunch of old events in each of the old OUs, so if you selected for example "QIS_test_int" and you're looking at the "Sent" list, then it's normal to see "Unrecognized" in old events. Old data not necessarily is valid, and we cannot guarrantee that the values were properly uploaded. Do you confirm that it's not in the "Sent" list where you're finding the "Unrecognized" issue, or being in the "Sent", that it's happening with values generated by this build (phone features with hash ending by 73d42f5c96dafd5f254867418c9d31bcfb3f548a)?

QISPSK commented 8 years ago

We've noticed a ton of unrecognized data for QIS test int on the "Sent" tab of the phone application. Oddly, none of this data appears to be in DHIS2, and it was all sent yesterday - a day that we had no training and thus, at least to my knowledge, no one was entering data aimed at the QIS test int OU.

Any idea why this might be showing up all of a sudden?

ifoche commented 8 years ago

@QISPSK (also mentioning @josemp10 to include him into this conversation). We can see a lof of blanks listed as day 21 in the server that do not exist app side. Our guess is that all those blanks were submitted by an Android device with the date set to 21 (so it was used 21 for establishing the eventDate), but it happened the 22. So I would say that here there are 2 things:

So our proposal if you agree the previous assessment. What would say if

Does it makes sense?

QISPSK commented 8 years ago

@ifoche Thanks for the detailed response! This does indeed make sense. I think your proposals sound like a good idea and we can keep an eye out for this issue in the future. We also agree that changing the listing sort order to the event date is preferable to the completion date, although (hopefully) there should only be a minor difference between them.

rodmelia commented 8 years ago

Sorry for the sharing changes - they occurred as Jose and I were fine tuning them.

ifoche commented 8 years ago

Can anyone confirm this is done?

QISPSK commented 8 years ago

@ifoche This issue is also unfortunately not fixed. We're still seeing unrecognized under qis_test_int. It may be associated with switching OUs during the trainings, but we aren't certain. Unrecognized is also likely showing up in the "Blank Data" issue submitted earlier today, but will confirm when we can contact the provider.

ifoche commented 8 years ago

Hi @QISPSK, I can confirm that this is probably only a consequence of the blank data bug. In fact, this is a protection against that. If this were not showing Unrecognized, the app would be crashing for not finding a valid option. So I would consider this not as a failure itself but as a consequence of blanks...don't you think? (thinking on closing the ticket)

QISPSK commented 8 years ago

I think closing this ticket is perfectly fine. Definitely seems like the issue is only linked to blank data and should resolve itself once that issue gets fixed.