[phenotips/patient-network] e491a6: PN-253: Create a configuration table to show the l...

GitHub noreply at github.com
Mon Jan 22 16:09:17 EST 2018


  Branch: refs/heads/stable-1.1.6
  Home:   https://github.com/phenotips/patient-network
  Commit: e491a6440c7e9cf9a567ac875b3d1d986cac82cb
      https://github.com/phenotips/patient-network/commit/e491a6440c7e9cf9a567ac875b3d1d986cac82cb
  Author: veronikaslc <veronika.koltunov at gmail.com>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    A ui/src/main/resources/PhenoTips/MatchingRunInfo.xml
    A ui/src/main/resources/PhenoTips/MatchingRunInfoClass.xml

  Log Message:
  -----------
  PN-253: Create a configuration table to show the last date+time matching was run


  Commit: 2e433767b03015ef5d4f6d62a37681afd58d89fd
      https://github.com/phenotips/patient-network/commit/2e433767b03015ef5d4f6d62a37681afd58d89fd
  Author: veronikaslc <veronika.koltunov at gmail.com>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/DefaultMatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/LocalMatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/internal/DefaultMatchingNotificationManager.java
    M pom.xml

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search


  Commit: bffc67f666bfb225eddbb0e3a7de089e82c194d6
      https://github.com/phenotips/patient-network/commit/bffc67f666bfb225eddbb0e3a7de089e82c194d6
  Author: veronikaslc <veronika.koltunov at gmail.com>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/DefaultMatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/LocalMatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/internal/DefaultMatchingNotificationManager.java
    M pom.xml

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search
Addressing codereview comments


  Commit: 63448ba54daa9d757daaa0c8541bd8d66b3fa50a
      https://github.com/phenotips/patient-network/commit/63448ba54daa9d757daaa0c8541bd8d66b3fa50a
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/LocalMatchFinder.java

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search

Added more logging and more error catching


  Commit: 8303f6c19662c313aad54e3682ca358b103a06be
      https://github.com/phenotips/patient-network/commit/8303f6c19662c313aad54e3682ca358b103a06be
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/DefaultMatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/internal/DefaultMatchingNotificationManager.java
    M pom.xml

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search

Simplified the API a bit


  Commit: d837ab09ceb1677341bcc408769fa51522509d42
      https://github.com/phenotips/patient-network/commit/d837ab09ceb1677341bcc408769fa51522509d42
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/DefaultMatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/LocalMatchFinder.java
    R ui/src/main/resources/PhenoTips/MatchingRunInfo.xml
    M ui/src/main/resources/PhenoTips/MatchingRunInfoClass.xml
    A ui/src/main/resources/PhenoTips/MatchingUpdateAndInfo.xml

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search

Added statistics about patients affected by the match


  Commit: 98f72b510470b239d0c9008894c18e2bb522fe32
      https://github.com/phenotips/patient-network/commit/98f72b510470b239d0c9008894c18e2bb522fe32
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinder.java
    A matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/AbstractMatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/LocalMatchFinder.java

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search

Refactored code
 - separated common matcher functionality into AbstractMatchFinder
 - extended MatchFinder API to provide more control over matching process


  Commit: d71316d37b3fd0ddf6137a7ef6823068c463179d
      https://github.com/phenotips/patient-network/commit/d71316d37b3fd0ddf6137a7ef6823068c463179d
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/MatchingNotificationManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/DefaultMatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/internal/DefaultMatchingNotificationManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/script/MatchingNotificationScriptService.java

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search
PN-243: Finding local and remote matches should have separate buttons

Back-end changes in various MatchingNotification components to allow better control over what matches are found
(local or remote, all or updated since last matcher run)


  Commit: 63ee144b931cd544922f522b10fc2631153232e5
      https://github.com/phenotips/patient-network/commit/63ee144b931cd544922f522b10fc2631153232e5
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M pom.xml

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search
PN-243: Finding local and remote matches should have separate buttons

pom.xml update


  Commit: 44c118c7c87c5c6de59084ffaf8aaed3e1b6f577
      https://github.com/phenotips/patient-network/commit/44c118c7c87c5c6de59084ffaf8aaed3e1b6f577
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/resources/ApplicationResources.properties
    A matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/findMatches.css
    A matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/findMatches.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/matchesTable.css
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/notifiedMatchesTable.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/unnotifiedMatchesTable.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/utils.js
    A matching-notification-ui/src/main/resources/MatchingNotification/MatchingRunInfoClass.xml
    A matching-notification-ui/src/main/resources/MatchingNotification/MatchingUpdateAndInfo.xml
    M matching-notification-ui/src/main/resources/MatchingNotification/RequestHandler.xml
    M matching-notification-ui/src/main/resources/MatchingNotification/UnnotifiedTable.xml
    R ui/src/main/resources/PhenoTips/MatchingRunInfoClass.xml
    R ui/src/main/resources/PhenoTips/MatchingUpdateAndInfo.xml

  Log Message:
  -----------
  PN-243: Finding local and remote matches should have separate buttons
PN-252: Run local matches search only for patients updated/created since last search

UI changes:
  - moved "find matches" button from the matching notification page to the "refresh matches" page
  - added separate buttons to find local matches and remore matches, as well as find all matches or only matches for updated patients
  - moved some .xml files around to have them in the same place with other code using common .js files


  Commit: 720da881cb444c2a6f22eb33c74b32130533e45f
      https://github.com/phenotips/patient-network/commit/720da881cb444c2a6f22eb33c74b32130533e45f
  Author: allasm <allasm at gmail.com>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/MatchingNotificationManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinderManager.java
    A matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/AbstractMatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/DefaultMatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/LocalMatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/internal/DefaultMatchingNotificationManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/script/MatchingNotificationScriptService.java
    M matching-notification-api/src/main/resources/ApplicationResources.properties
    A matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/findMatches.css
    A matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/findMatches.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/matchesTable.css
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/notifiedMatchesTable.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/unnotifiedMatchesTable.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/utils.js
    A matching-notification-ui/src/main/resources/MatchingNotification/MatchingRunInfoClass.xml
    A matching-notification-ui/src/main/resources/MatchingNotification/MatchingUpdateAndInfo.xml
    M matching-notification-ui/src/main/resources/MatchingNotification/RequestHandler.xml
    M matching-notification-ui/src/main/resources/MatchingNotification/UnnotifiedTable.xml
    M pom.xml

  Log Message:
  -----------
  Merge pull request #169 from phenotips/PN-253

PN-253: Create a configuration table to show the last date+time matching was run; PN-252: Run local matches search only for patients updated/created since last search; PN-243: Finding local and remote matches should have separate buttons


Compare: https://github.com/phenotips/patient-network/compare/9d6048091174...720da881cb44


More information about the notifications mailing list