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

Remove include param from action request #128

Merged
merged 1 commit into from Nov 7, 2019

Conversation

@paulfalgout
Copy link
Member

paulfalgout commented Nov 7, 2019

Backend will enforce include. Patient models are still included, but by default.

This is such a nothing change, if tests pass I'll merge it

Backend will enforce include.  Patient models are still included, but by default.
@paulfalgout paulfalgout merged commit 5a5825c into master Nov 7, 2019
5 checks passed
5 checks passed
Automated Preview AWS pull request preview
Details
codecov/patch 100% of diff hit (target 98.26%)
Details
codecov/project Absolute coverage decreased by -4.71% but relative coverage increased by +1.73% compared to 1912cf7
Details
cypress: care ops 61 tests passed in 00:57
Details
test-deploy Workflow: test-deploy
Details
@paulfalgout paulfalgout deleted the chore/remove-action-include branch Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.