Note |
---|
The outcome of this project can be seen here: Feedback Module |
Primary mentor | |
Backup mentor | |
Assigned to |
Table of Contents | ||||
---|---|---|---|---|
|
...
Ideally, this is ubiquitously available in the interface in a simple & intuitive format – e.g., a "Feedback" link (or button) or an option under "Help" like "Send Feedback" that pops up a simple form. When feedback is submitted, the user receives confirmation and the feedback is forwarded to the system administrator.
Interested Parties
- Unlicensed user
- Unlicensed user
- Unlicensed user
- Unlicensed userBurke Mamlin
- Mike Seaton
- Darius Jazayeri
- Ben Wolfe
Required Skills
- Experience with java programming
- Familiarity with JSP and servlet programming
...
- You have a bordered box in a popup dialog, which gives unnecessary nested borders. Get rid of the box. Done : http://twitpic.com/5xf02w/full .
- Allow multiple comments for a feedback item : Done.
- Allow admin to set an e-mail address to receive notifications of new feedback entries – e.g., the admin may wish to receive an e-mail with convenient links to act on the feedback (e.g., single-click options to "delete", "-mark as duplicate", "forward to OpenMRS as bug", etc.)- : Done , screenshots : http://twitpic.com/5xrtr9/full
- Allowing for the simplest use case of clicking a feedback button, writing a comment in a textarea, and submitting – i.e., provide a default for other fields or make them optional. ( Source : Link ) : Done
- No need for Index column in predefined subject edit form ... Done , screenshots : http://twitpic.com/5xj1oo/full , http://twitpic.com/5xj1sh/full , http://twitpic.com/5xj1vo/full
- Why are times not being stored? I only see dates for what should be a datetime column Done: Now changed to the timestamp in liquibase.xml file : http://twitpic.com/5xg0ml
- Rather than "Add Severity" and "Add Status" perhaps use "Severities" and "Statuses" ... "Stati"? Done , screenshots : http://twitpic.com/5xjho2/full and http://twitpic.com/5xjhpn/full .
- In case you add a severity after the module has been running for a while, you should be able to put it in the middle of the list, where it belongs. (Same for status.) (adding a "sort_weight" column to each of the two tables?) Done will be seen as of very next commit after commit 22100 .
- "Creator" should probably display the default PersonName rather than userid ... unless that is a requirement. : Done : http://twitpic.com/5xgc02/full
- It would be nice for the admin to be able to define what text is shown to the user after they submit. It could just be stored as a global property. - Ben Wolfe ( https://source.openmrs.org/cru/CR-MOD-202#CFR-9188 )
- In Manage Feedback, change Date Submitted to age (e.g. "yesterday"). Added Support to : http://ocpsoft.com/prettytime/ , screenshot : http://twitpic.com/5xit6d/full is this okey ?
- Perhaps show example of "default" selections for subject and severity level. : Done.
- Can the simple edit forms be done in a popup dialogs? (same for other one-liner phrases) Done.
- Provide a simple mechanism for the admin(s) to forward feedback to OpenMRS when appropriate (e.g., bugs reports) – i.e., after removing or ensuring there is no identifiable patient information in the content. Done.
- Allow the admin to edit templates for e-mail notifications (both confirmation to user and notification to admin)
...