Chart Search Module
  1. Chart Search Module
  2. CSM-89

Re-factor the module's source code to remove any redundant code

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: TBD TBD
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
    • Complexity:
      Undetermined

      Description

      Re-factor the module's source code to remove any redundant code, classes and libraries (if there are any) introduced as a result of the previous module evolution.

        Gliffy Diagrams

          Activity

          Hide
          Nicholas Folk added a comment -

          Thank you, this will be very helpful. I have also found some more content to add to the message.properties file which I will push shortly. I have been on vacation for the weekend so I haven't had much time, but I will take a look at your comments and work on them over the next few days.

          Show
          Nicholas Folk added a comment - Thank you, this will be very helpful. I have also found some more content to add to the message.properties file which I will push shortly. I have been on vacation for the weekend so I haven't had much time, but I will take a look at your comments and work on them over the next few days.
          Hide
          Nicholas Folk added a comment -

          here is another pull request with just a few more messages added to the messages.properties file.

          https://github.com/openmrs/openmrs-module-chartsearch/pull/73

          Show
          Nicholas Folk added a comment - here is another pull request with just a few more messages added to the messages.properties file. https://github.com/openmrs/openmrs-module-chartsearch/pull/73
          Hide
          Kaweesi Joseph added a comment -

          Merged at: e65c397

          Thanks Nicholas Folk

          Show
          Kaweesi Joseph added a comment - Merged at: e65c397 Thanks Nicholas Folk
          Hide
          Nicholas Folk added a comment -

          No problem Kaweesi Joseph, do you have anything else specificly in mind that I can refactor at the moment?

          Show
          Nicholas Folk added a comment - No problem Kaweesi Joseph , do you have anything else specificly in mind that I can refactor at the moment?
          Hide
          Nicholas Folk added a comment -
          Show
          Nicholas Folk added a comment - Meanwhile, here's another small refactoring Kaweesi Joseph : https://github.com/openmrs/openmrs-module-chartsearch/pull/74

            People

            • Assignee:
              Nicholas Folk
              Reporter:
              Kaweesi Joseph
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 2 days
                2d
                Remaining:
                Remaining Estimate - 2 days
                2d
                Logged:
                Time Spent - Not Specified
                Not Specified

                  Development