Skip to content
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.

Sign up for GitHub

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

Jump to bottom

UI: Merge two Patients in external Archives #960

Closed
gunterze opened this issue Sep 14, 2017 · 2 comments
Closed

UI: Merge two Patients in external Archives #960

gunterze opened this issue Sep 14, 2017 · 2 comments
Assignees
@shral
Labels
enhancement New feature or request
Milestone
5.10.6

Comments

@gunterze
Copy link
Member

Utilizing /hl7apps/{appName}/hl7/{externalAppName}/patients/{priorPatientID}/merge
Merge Patient external

Use HL7Application Name associated with with local and remote AE as Sending and Receiving Application Name. s. #957
@gunterze gunterze added this to the 5.10.6 milestone Sep 14, 2017
shral added a commit that referenced this issue Sep 25, 2017
@shral
[UI: Merge two Patients in external Archives #960]
9e05298
@shral
Copy link
Member

shral commented Sep 29, 2017

Rest URL is prepared, but we should send to the server the whole DICOM object not the object with custom defined fields like we have for internal patient merge.

@shral shral closed this as completed Sep 29, 2017
@vrindanayak vrindanayak reopened this Oct 4, 2017
@vrindanayak
Copy link
Member

  • Target Patient ID being sent in URL instead of Prior Patient ID
  • Request body contains json in format mentioned for Internal merge causing JsonParsingException

shral added a commit that referenced this issue Oct 6, 2017
@shral
adding patient object in select, finishin external merge of patients[…
5414ee0 
…UI: Merge two Patients in external Archives #960]
@shral shral closed this as completed Oct 6, 2017
shral added a commit that referenced this issue Oct 6, 2017
@shral
adding warning if the user tryes to merge multiple patients[UI: Merge…
fc23ed0 
… two Patients in external Archives #960]
shral added a commit that referenced this issue Oct 6, 2017
@shral
refactorizing[UI: Merge two Patients in external Archives #960]
67cff50
@vrindanayak vrindanayak mentioned this issue Oct 11, 2017
Closed
@vrindanayak vrindanayak mentioned this issue Nov 23, 2017
Closed
@gunterze gunterze added enhancement New feature or request and removed new feature labels Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shral shral

Labels
enhancement New feature or request
Projects
None yet
Milestone
5.10.6
Development

No branches or pull requests
3 participants
@gunterze @shral @vrindanayak