-
Notifications
You must be signed in to change notification settings - Fork 360
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
refactor: align backend response status with UI #4433
refactor: align backend response status with UI #4433
Conversation
a122411
to
cfc5a07
Compare
Is not possible to do a data migration updating old |
The URL of the deployed environment for this PR is https://argilla-quickstart-pr-4433-ki24f765kq-no.a.run.app |
Hi Team! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #4433 +/- ##
===========================================
+ Coverage 90.13% 91.33% +1.19%
===========================================
Files 233 335 +102
Lines 12493 19326 +6833
===========================================
+ Hits 11261 17651 +6390
- Misses 1232 1675 +443
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Hi guys! the CI is not working due to some failed tests, could you take a look? |
…ub.com:argilla-io/argilla into refactor/align-backend-response-status-with-UI
The |
Description
After changes introduced in #4346, the pending status represents exactly what the
missing
status is. This PR deprecates themissing
status for record responses in favor of a newpending
status.@damianpumar, maybe you can adat the changes in the frontend part.
Refs: #4346
Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)
Checklist
CHANGELOG.md
file (See https://keepachangelog.com/)