Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • OPEN (Post potential topics of interest on Talk with the tag design-forum)Review next meeting agenda
  • Attendees
    • Darius
    • Wyclif
    • Burke
    Agenda
    • Exanded cohorts update
    Notes
    Reviewed expanded cohorts status <https://issues.openmrs.org/browse/TRUNK-4906>
    Clearer (more meaningful) exceptions for order entry <https://issues.openmrs.org/browse/TRUNK-4696>
    Summary of To Dos:
    • Move all exceptions from org.openmrs.api.order.exceptions.* under org.openmrs.api.*
    • Remove org.openmrs.api.order and org.openmrs.api.order.exception packages
    • AmbiguousOrderException should extend OrderEntryException
    • Backport to 2.1.x (also backport one other commit made already)
    Where should the exception live?
    Example: org.openmrs.api.order.exception.CannotStopInactiveOrderException
    should we drop the "api"?
    should we drop the "exception"?
    TODO: Stick with our current (not ideal) convention, and move all of these to org.openmrs.api
    TODO: AmbiguousOrderException should extend OrderEntryException

Notes

 

Transcripts

  • Audio recording of the call: Listen online or download (available after the meeting)