-
Notifications
You must be signed in to change notification settings - Fork 401
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
[FEATURE] SDK - Add support for response status #4977
[FEATURE] SDK - Add support for response status #4977
Conversation
Also, the inner userRespnose resource manage the whole status based only in provided answers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a low comment on warning.
73448d6
to
6ffdd59
Compare
e4db95f
to
59a8b94
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/v2.0.0 #4977 +/- ##
================================================
+ Coverage 60.92% 91.32% +30.40%
================================================
Files 329 136 -193
Lines 17674 5833 -11841
================================================
- Hits 10767 5327 -5440
+ Misses 6907 506 -6401
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Description
This PR adds support to provide the status for record responses. This is quite important to admin users when restoring annotated data from external sources. Also, the migration script from the legacy dataset can benefit the change.
When several statuses are found for the same user before sending data to the server, a warning is shown to the users and the
draft
status is selected.Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)
How Has This Been Tested
(Please describe the tests that you ran to verify your changes. And ideally, reference
tests
)Checklist
CHANGELOG.md
file (See https://keepachangelog.com/)