opensrp / opensrp-client-tbr

Other
2 stars 4 forks source link

Update the list of configurable columns for registers #46

Closed cafootitt closed 6 years ago

cafootitt commented 6 years ago

Suggestions from the 7 Dec call:

Once the mockups are updated for any-register, presumptive-only, positive-only, and in-treatment only, send to IHS for annotation.

rowo commented 6 years ago

@cagulas these below look like they cover all of the updates? columns-any columns-presump-pos columns-intreatment

cafootitt commented 6 years ago

For presumptive diagnose column, I don't think encounter would be a common term used (it comes from OpenMRS, so the "Enc" might be confusing to people. If this is the screening date, maybe we can abbreviate to "Scr" or something like that?

@alihabib @kwasim thoughts?

cafootitt commented 6 years ago

@alihabib @kwasim For the Treatment column, there is no field in the xlsform where the user inputs intensive vs. continuation, so I'm assuming this is based on a calculation of how long they've been on treatment? What is that automated calculation?

Also I note that the Regimen field in the form is a text field. Are we not having a dropdown list of medication regimens?

cafootitt commented 6 years ago

@rowo for the In-treatment results column, is this intended to show only 1 latest result at a time? So if they have two results entered on the same day, only the last one entered is shown?

rowo commented 6 years ago

For first encounter date, I just used Google, https://www.allacronyms.com/encounter/abbreviated. I'm good with whatever IHS says.

For latest result, yes there is kind of only room for one result. I guess it would have to be last one entered. Other options are: put two results if they happen on the same day b/c the Xpert test won't be happening and so two results may fit or prioritize them in an order to show up if they happen on the same day. I don't really know the technical implications, but in general I think the last result is okay because it indicates the last time a result came in and generally what it was, but no one would ever make any decisions based solely on that and so would open the detail view. CG: Makes sense, plus, the latest discussions we had indicate that commonly, patients only have repeat smear results taken during treatment phase.

cafootitt commented 6 years ago

@alihabib @kwasim Once we get feedback on the couple of questions above, I'll go ahead and create implementation issues to update the devs on all of the configurable columns throughout the app. Let us know if all of the above looks ok.

kwasim commented 6 years ago

for presumptive diagnose column, screening date makes sense to me as suggested by @cagulas which can be abbreviated to "scr date". It may fit into rows but I think that's okay.

for TB phases, for drug susceptible tb, first two months is considered an intensive phase followed by 4 months of continuation phase. I am not sure though if implementation varies from country to country.

as for the regimen, I also think it should be a dropdown.

for Diagnose, diagnosis, results etc columns, I think showing dates (day/month) would be a better idea than say xd ago or xm ago @cagulas @alihabib what you guys think?

u-moosajee commented 6 years ago

@cagulas @rowo are these the updated mockups that need to be annotated?

cafootitt commented 6 years ago

@kwasim ok great, thanks.

@u-moosajee I think Roger will post one more update, so maybe it's a good idea to wait for the final versions before annotating them? On a separate note, would you be able to update the xlsform to make the treatment regimen a dropdown with regimen options instead of a text field?

@alihabib Should we automatically calculate intensive vs. continuation phase (first 2 months, then remaining 4 months) or would this vary by site?

rowo commented 6 years ago

@kwasim just to confirm, did you do "scr date" lowercase on purpose? RW: wasim said not on purpose

@kwasim any final decision on date format? RW: on call, MM/DD/YYYY was final decision

I also came across an old issue, https://github.com/OpenSRP/tbr5/issues/47#issuecomment-337903427, where @alihabib recommends a schedule column/button for Smear that I never drew up — is this still applicable @kwasim @alihabib ? RW: smear schedule column should be added

rowo commented 6 years ago

I made the following updates:

col-1

col-2

col-3

u-moosajee commented 6 years ago

@rowo hi roger, please view the attached annotated images for the register columns

@alihabib could you please advise on what dropdown options to add for treatment regimen as suggested by Caroyln above

anyregcolumn_annotated intreatmentreg_annotated specificregcolumns_annotated

githengi commented 6 years ago

@alihabib Will the Smear Result due date be added to the Smear Result form?

cafootitt commented 6 years ago

@u-moosajee can you please update the mockups with the variable names for the follow up and smear result schedules in the forms? Also the treatment column the phase of treatment and treatment regimen. Thanks.

u-moosajee commented 6 years ago

tbr5 annotated image

@cagulas please see the annotated mockup with the missing variable names incorporated

cafootitt commented 6 years ago

Found a few issues with the columns, but I'll create a new issue and close this issue.