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.

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

Adding validation on $export type filters. #4617

Merged
merged 3 commits into from
Sep 19, 2024

Conversation

v-iyamauchi
Copy link
Contributor

Description

Describe the changes in this PR.

Related issues

Addresses [issue #121238].

User Story 121238: Export job behavior when search criteria does not match

Testing

Tested by manual testing and adding automated test cases.

FHIR Team Checklist

  • Update the title of the PR to be succinct and less than 65 characters
  • Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
  • Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
  • Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
  • Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@v-iyamauchi v-iyamauchi added Bug Bug bug bug. Open source This change is only relevant to the OSS code or release. labels Sep 16, 2024
@v-iyamauchi v-iyamauchi added this to the S150 milestone Sep 16, 2024
@v-iyamauchi v-iyamauchi requested a review from a team as a code owner September 16, 2024 21:04
@v-iyamauchi v-iyamauchi merged commit 1c09493 into main Sep 19, 2024
44 of 46 checks passed
@v-iyamauchi v-iyamauchi deleted the personal/v-iyamauchi/add-exportfiltervalidation branch September 19, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bug bug bug. Open source This change is only relevant to the OSS code or release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants