New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PTM-86 : Remove matching pairs from the report when patients are updated #38

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@Lahiru-J
Member

Lahiru-J commented Aug 8, 2017

In the incremental matching if the already generated report would affect by updating a patient then the report too should be updated. For example,
Suppose there is a matched pair (say patientA and patientB). User updates the patientA in a way that patientA no longer exhibits matching properties with patientB. Then the matched pair in the report should be removed.

@raghavdhingra

Looks nice to much extent.

@@ -0,0 +1,68 @@
package org.regenstrief.linkage.io;

This comment has been minimized.

@Ben-Waters

Ben-Waters Apr 14, 2018

Missing the license text at the top of this file

DWRMatchingConfigUtilities.doAnalysis(blListStr);
if(actRightAway=="true"){
var checkBoxIncremental = document.getElementById("cbIncrementalMatch");
DWRMatchingConfigUtilities.doAnalysis(blListStr, checkBoxIncremental.checked);

This comment has been minimized.

@Ben-Waters

Ben-Waters Apr 14, 2018

The tabbing on this section of code looks incorrect.

@dkayiwa

This comment has been minimized.

Member

dkayiwa commented Jun 20, 2018

@Lahiru-J are you still interested in finishing this up?

@dkayiwa dkayiwa closed this Jul 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment