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

Bulk Export Status should contain the required "error" field #2715

Closed
dabimahesh opened this issue Aug 24, 2021 · 2 comments
Closed

Bulk Export Status should contain the required "error" field #2715

dabimahesh opened this issue Aug 24, 2021 · 2 comments
Assignees
@prb112
Labels
enhancement New feature or request
Milestone
Sprint 2021-11

Comments

@dabimahesh
Copy link

While doing bulkdata status request "error":[], should be a part of the response, even if there is no error in this request. Please help us in this. This is required for ONC MU Inferno testing tool.

https://chat.fhir.org/#narrow/stream/212434-ibm/topic/releases/near/250080961

Thanks
Mahesh
@prb112 prb112 added the enhancement New feature or request label Aug 24, 2021
@prb112
Copy link
Contributor

prb112 commented Aug 24, 2021 •
edited
Loading

Thanks, marked this is an enhancement.

@prb112 prb112 self-assigned this Aug 24, 2021
@prb112 prb112 added this to the Sprint 2021-11 milestone Aug 24, 2021
prb112 added a commit that referenced this issue Aug 24, 2021
@prb112
Minor Fixes
eb64e93 
- Bulk Export Status #2715
	- Added a default Error
- Bulk Export Status Report #2714
	- Added property for requiresAccessToken
- Composition reports a 500 when it's a bad user request #2717
	- Mutate 500 to 400 in $document
	- Add Test for $document when bad request

Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
@prb112 prb112 mentioned this issue Aug 24, 2021
Merged
@lmsurpre
Copy link
Member

lmsurpre commented Sep 3, 2021

Confirmed that, upon successful export, the bulkdata-status endpoint response now contains "error": []

@lmsurpre lmsurpre closed this as completed Sep 3, 2021
@lmsurpre lmsurpre changed the title Bulk Export Status Bulk Export Status should contain the required "error" field Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prb112 prb112

Labels
enhancement New feature or request
Projects
None yet
Milestone
Sprint 2021-11
Development

No branches or pull requests
3 participants
@dabimahesh @prb112 @lmsurpre