Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Migrated to Confluence 5.3
Note

The outcome of this project can be seen here: Feedback Module

Primary mentor

Unlicensed user Ben Wolfe

Backup mentor

Unlicensed userBurke Mamlin

Assigned to

Unlicensed user Gaurav Paliwal

Table of Contents
minLevel2
styledisc

...

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

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 entriese.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)

...