Unsorted feedback:
-
There’s a Having Trouble With Login link on the login screen. It takes you to OpenMRS Dictionary Manager which mentions getting a token, which is very very out of date. There’s also a video, which I didn’t watch, but is probably old too. Either remove this link, or else replace the content of the help page with something current.
-
I do not see the login page bug that Andy is experiencing (i.e. I don’t immediately see a “logging in” with no ability to do anything.)
-
The username and password given in this thread do not work for me. Maybe this server’s database gets regularly reset? (I’m able to create a new account though, although it took >5 minutes to get the confirmation email.)
-
I click Log out, and the flash message is missing a space.
-
Create New Dictionary screen:
- As mentioned on the call today, reorder the fields when creating a new dictionary, as: name, short code, description, preferred source, owner, visibility, default language, other languages. (MVP)
- “Start by copying another dictionary” => this feature isn’t implemented yet is it? If not, then remove this field
- This is too big to be a popup. I have to scroll to submit the form in a full-screen browser on a 15" laptop. It should be its own screen. (post-MVP)
-
General Details → Edit
- I edited the dictionary and added some other languages. I click update dictionary. I immediately click Edit again, and the popup does not show my additional languages. If I refresh the page and click edit, I do see them. Clicking around it appears maybe the bug is if I click edit 2 times without refreshing the page, the edit screen doesn’t have the latest info. (MVP)
-
Dictionary Concepts
- with an empty dictionary the left side filters just show their headings. But it should say “none” after each of Sources and Class (post-MVP)
- change “Sources” to “Source” so it’s consistently pluralized with “Class”
- Need to be able to read the whole name. I don’t care exactly how you fix this (wider column, wrapping text, etc), but it’s MVP. This doesn’t work:
- No way to view a referenced concept. Quick fix is to add a “View in Traditional OCL” link under Actions. MVP.
- No way to see a concept’s ID. It’s probably okay to show this in the source column (e.g. “CIEL - 123”). Another option would be to change the first column to be “Concept” and then have it be ID - Name, like I think we’ve done elsewhere. (MVP)
-
Add CIEL concepts
- As Andy mentioned on the call today, when you first open this, do not trigger an empty search, and do not show any concepts. (There are 60000 concepts, so showing one page is not helpful.)
- Name must be fully displayed in the search results. Don’t care what fix you choose (be consistent with the other screen though) but it’s MVP. E.g. this view doesn’t work:
- clicking a row in the table should open the preview (post-MVP; and let’s rediscuss before you actually do this)
- in the previous example (“pulmonary tuberculosis”) the search results should show the matched name (post-MVP, requires coordination with backend team)
- I searched for 1268 (which I actually already added to my dictionary, and the first results are 126800, 126801, … This is bad behavior, but I don’t know which it is:
- If something is already in the dictionary, we should still show it in the search results. (Okay to gray it out, or disable the add button.)
- Exact match on concept ID should show be the first result. This may be a back-end fix, but if this is the scenario, please create a ticket to track it. Make it an MVP ticket for now, though we could punt it.
- When I add a CIEL concept that’s a q-and-a, this should also add the answer concepts to my dictionary, but this didn’t happen. (MVP) Example: https://demo.openconceptlab.org/orgs/CIEL/sources/CIEL/concepts/1268/
- Similarly, if you add a Set, it should automatically add its members. (MVP)
- (Inverse mappings should not automatically be added, e.g. if I add the answer it should not automatically add the question.)
-
Creating concepts:
- As mentioned on the call, Create Another Type of Concept needs to have all fields available (MVP)
- As mentioned on the call, Create A Set Of Concepts needs to actually let you specify set members (MVP)
- For all Create options that let you choose from multiple Class options, the default selection should be blank (i.e. we don’t expect that the first option in the dropdown is usually correct). E.g. Set of concepts, Symptom/Finding,
- I did the following series of steps and found a strange bug:
- steps:
- create a Symptom/Finding
- Name: “Itchy Nose” in English, preferred in language = true
- Clicked add another name
- Chose the language French for the other name
- Clicked Remove on the second name without entering anything
- Entered description
- Save Concept
- bug(s)
- it took me back to the Dictionary Concepts screen, but did not show my new result.
- when I refresh the page, I see a row that’s empty
- when I click Edit on that one, it shows the empty Name row (that I removed)
- expected behavior: removing a name should really remove it, not just hide it in the UI (MVP)
- when I try the same thing with Spanish instead of French, the same bug occurs.
- steps:
- Another bug: when I update it to include an actual name in French, this is displayed on the Dictionary Concepts screen (even though my dictionary’s default language is English). (MVP)
- when I try this same thing with Spanish instead of French, this bug did not occur. (I don’t know if that’s random, or because French was listed in Other Languages for my dictionary.)
- The Diagnosis and Procedure concepts that I created show their Class with the word starting in lowercase; when I add a CIEL concept, or create another kind of concept, its name starts in uppercase. The value we store here MUST be equal to one of the expected values, so this will probably fail to import into OpenMRS. (MVP)
That’s all I have time for now, will try to pick this up again later.