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

Call on record update service #182

Closed
kostobog opened this issue Jul 11, 2024 · 1 comment
Closed

Call on record update service #182

kostobog opened this issue Jul 11, 2024 · 1 comment
Assignees
@kostobog

Comments

@kostobog
Copy link

kostobog commented Jul 11, 2024 •
edited
Loading

@blcham
Call external service on record update
A/C :

  • onRecordUpdateServiceUrl - record manager can be configured which service to call
  • record manager calls the onRecordUpdateServiceUrl with the updated record URI on successful PatientRecordController.updateRecord
  • if onRecordUpdateServiceUrl is not configured do nothing
@kostobog kostobog self-assigned this Jul 11, 2024
kostobog added a commit to kbss-cvut/record-manager that referenced this issue Jul 11, 2024
@kostobog
[Fix partially kbss-cvut/record-manager-ui#182] publishRecords does n…
0f9bd43 
…othing when publishServiceUrl is not configured.
kostobog added a commit to kbss-cvut/record-manager that referenced this issue Jul 11, 2024
@kostobog
[Fix partially kbss-cvut/record-manager-ui#182] Call onRecordUpdateSe…
66f5b96 
…rviceUrl if configured when record is updated.
kostobog added a commit to kbss-cvut/record-manager that referenced this issue Jul 11, 2024
@kostobog
[Fix partially kbss-cvut/record-manager-ui#182] Call onRecordUpdateSe…
ba41ba1 
…rviceUrl if configured when record is updated.
@kostobog kostobog mentioned this issue Jul 11, 2024
Merged
kostobog added a commit to kbss-cvut/record-manager that referenced this issue Jul 11, 2024
@kostobog
[Fix partially kbss-cvut/record-manager-ui#182] Call onRecordUpdateSe…
6761b69 
…rviceUrl if configured when record is updated.
kostobog added a commit to kbss-cvut/record-manager that referenced this issue Jul 11, 2024
@kostobog
[Fix partially kbss-cvut/record-manager-ui#182] Fix failing test
f91a828
@blcham
Copy link

blcham commented Jul 12, 2024

Implemented by kbss-cvut/record-manager#57

@blcham blcham closed this as completed Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kostobog kostobog

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@blcham @kostobog