OCL for OpenMRS User Feedback discussion

Thanks @burke, we’ll be looking at each of these. The message has also been updated to address the earlier confusion and to include a wiki where you can track your feedback.

Assorted feedback, from my initial testing:

MVP:

  • Create dictionary
    • gives two options for Preferred Source (CIEL, PIH) but I think the application only really supports CIEL, right? So, remove the PIH option.
    • there’s an option for “Start by copying another dictionary”, but it doesn’t work right. (I tried to copy a dictionary that has 13 concepts, and the resulting dictionary only has 10 concepts.) Please hide this feature for MVP release, and create a ticket about making it work right later.
  • Dictionary Overview
    • The “By class” breakdown isn’t working right. It shows “Diagnosis: 0” when I have a lot of diagnoses. Fix this if it’s quick (e.g. if it’s just a question of changing “diagnosis” to “Diagnosis” in a query) but if it’s going to take a lot of work, just remove this breakdown for the MVP release.
  • Concepts -> Create Custom
    • I try to create a concept with two names in English, the first one has type=Fully Specified, the second has type=Synonym. I get “An error occurred when creating a concept. Invalid name type” for the one that’s a synonym. (Possibly OpenMRS calls this “synonym” but OCL backend calls this nametype=null.)

Post-MVP:

  • Dictionary Overview -> General Details
    • shows “public access: view”. This should instead be “visibility: public”. (I.e. it should be formatted for display the same way that the create screen does it.)
    • should also show the languages. (Can combine default language and other languages to one line for display.)
  • Dictionary Overview -> Actions
    • disable the Release button if there are zero concepts in the dictionary
  • Concepts -> Add from CIEL
    • If I add a concept, it does automatically add Answers and Set Members. This is good. But the flash message should also tell the user that this happened. E.g. “Added ANTENATAL VISIT REASON, and 2 concepts that it depends on.” (maybe even give the names of the extra concepts added).
    • If the user manually removes a concept that was added for being a dependency, I think this will break the import. (E.g. I added ANTENATAL VISIT REASON but then I remove PLANNING PREGNANCY.) We should block the user from removing any concept that’s a dependency of others. (I suspect this is a bit tricky, so marking it post-MVP.)
  • Concepts -> Bulk Add
    • why does the textarea have a loading spinner the first time I open this page?
    • this page needs to block you from navigating away while it’s doing the bulk add (otherwise in-progress work gets lost). Even better is to have some sort of progress report about what’s going on.
    • I added “1, 2, 3, 4, 5”. It told me that 5 and 3 were invalid (good), but then it told me “15 concepts added”, even though it actually only added 3 concepts. Same thing happened with concepts 6,7,8,9,10, there were also two invalid yet it said it was adding 15 concepts.
  • Concepts -> Create Custom
    • language dropdowns for Name and Description should only include supported languages.
    • should have better defaults for name type and preferred-in-language when you add a second name. (i.e. you’re only allowed one fully-specified, and one preferred-in-language, so if the first row has those, the second row should default to Synonym and No)
  • Concepts -> Edit
    • when editing, since OpenMRS UUID and OCL ID are not editable, make them take less space, and get rid of the help text.

Hello community, The testing phase is still ongoing, through to the 26th June. Please log on and test out the application.

Hello community,

We shall be running a final testing period for this round of development for the OCL client MVP this week(8th - 14th July 2019).

Please follow this link to test the application. Key pointers:

  • If you do not have credentials, use the old OCL to sign up, and then be sure to come back to the new OCL, again here.
  • If you are unable to login even after you had signed up, the database may have been reset and you would have to sign up again

A more detailed guide can be found on the wiki, here.

All feedback so far and progress/ resolution on it can be tracked on this wiki.

cc @dkayiwa @akanter @darius @c.antwi @ball @ruffjm @paynejd

I got a bit more time to test just now.

About creating/editing dictionaries

  • I created a dictionary. Then I click Edit.
    • and the owner dropdown looks like this: image
    • then I change visibility to Private, and the owner dropdown changes to this: image
      1. It should not show two options where there’s only real option
      1. It shouldn’t change the text for no apparent reason.
    • I do not think this is an MVP bug, though I’m concerned, especially given the next thing I did…
  • Then I did something random (closed that dictionary, clicked New Dictionary, then Cancel, then open the first dictionary again and click Edit) and now it looks like it lost a lot of values:
    • the owner dropdown looks like this: image
    • this is an MVP bug. It seems like state is being improperly shared across different instances of opening this form, and this makes me concerned about the underlying code.

About creating concepts:

  • I added “Reasons for Referral” to my dictionary. First it does a nice-looking thing where it shows that it’s going to add 39 dependent concepts, then the number goes up to 42. But then it just says “Finalizing” and it waits there for a long time.
    • That text is not helpful. I would change the initial text to say “Found X dependent concepts to add” and then “Adding and dependent concepts. This may take several minutes. DO NOT NAVIGATE AWAY FROM THE APPLICATION UNTIL THIS IS COMPLETE.”
    • I notice that I was able to navigate away from the Add screen and it seemed like things kept working, which is nice. (If this is not the case, then we need to prevent navigating away.)
    • I left the tab and when I came back to it the message was gone and I didn’t see any of the new concepts in my dictionary. I did a browser refresh and then I could see them. (This is MVP, that when the long-running task completes and you’re looking at the dictionary, it must refresh to show you the new state.)
  • At this point one of my concepts is being displayed with a French name, and everything else with English:
  • I try to create a concept with the same name as an existing concept that I already added from CIEL (“Night Blindness”) and the UI never gives me any indication that there was an error. (It showed a yellow message, then refreshed the concept list.). The result was that if I look on the back end my Source did have a concept created, but this was not added to my Collection.
    • I think this is an MVP to fix (i.e. don’t want to ever end up in the state where there is some concept in our Source, but not in the Collection.)
    • Or did we already discuss this and decide it was too hard to fix now?
    • The additional confusing thing for the user is that because of this I have used up an OCL ID, though I cannot see this in the new UI. So if I try to create my next concept with id 1 (since there’s nothing with that ID in the collection yet) I get a different error message that’s hard to interpret, and it disappears after ~5 seconds, which wouldn’t be long enough if I didn’t already know what’s going on. And then whatever I entered is lost. (“An error occurred when creating a concept. Concept with this Mnemonic and Parent id already exists. for all”)

Darius, there are issues with OCL showing the French as the preferred name in the UI. I don’t know why we do the same if we are viewing our data. If we are only viewing what is shown in OCL, then the same issue is there too.

I found some cosmetic issues:

  1. Dictionary description doesn’t wrap:

And the first time I open the dictionary I see Source0, Class0:

1562618131267blob.png

Hey @darius, @akanter ,

Localization is an open issue(enhancement) here(Concept display name should be populated dynamically based on the requesting user's locale preferences and the attributes of a concept name · Issue #15 · OpenConceptLab/ocl_issues · GitHub). Currently, the display name is the first name in the list of names.

Since we can’t cross validate across collections and sources, the way I see it, we have two options here;

Yes, the ticket was MVP+(Log in with Atlassian account)

The rest of the items that have been pointed out are being worked on.

cc. @dkayiwa @c.antwi

@karuhanga any update with progress on the above?

Hello @c.antwi,

Kindly refer to this wiki for the resolution on each of these.

And These Test Cases?

Has anyone managed to test from the collection down to an OpenMRS implementation via Subscription module?

Yes, I’ve been testing this out for a while and it behaves correctly.

@karuhanga @akanter @herbert24 @darius Should we continue to use this Talk thread for user feedback?

I’ve been testing OCL for OpenMRS Version 2. Overall it looks good. Nice work @karuhanga.

I found these 2 things – only the first issue is major:

Mappings: Not possible to select a source (or add a mapping).

Navigating: Confusing about navigating up/down. Maybe hover text could help? What is the arrow in the upper right corner when adding a concept?

Look/feel improvement (minor). This should be single-spaced: image

I also thought that there were insufficient sources available for mapping. Do we need to load the database that we were using for QA with this code, or better yet, move the code to the QA database?

Is this for all sources? Or this down to Andrew’s point about insufficient sources? Can you find CIEL for example?

This is pointing to qa. Perhaps we could it point it to demo instead?

No sources appear. Nothing in the dropdown. Nothing that can be searched.

I’d appreciate it if you did a screen grab searching ‘CIEL’

I see the behavior, but it’s not intuitive.

Search for ‘CIEL’ and (enter): image

Search for ‘ICD’ and (enter), and nothing happens: image

The previous version of OCL for OpenMRS had a dropdown with all the sources. That seems better, but willing to discuss other choices.

1 Like

I like the idea about having a combo box behavior where I see the top ones easily but can sponge search the list.