-
Notifications
You must be signed in to change notification settings - Fork 3.1k
feat: FIT-750: Update Agreement Selected UI #8575
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
base: develop
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for heartex-docs canceled.
|
✅ Deploy Preview for label-studio-storybook ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
✅ Deploy Preview for label-studio-playground ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #8575 +/- ##
===========================================
- Coverage 67.42% 67.41% -0.02%
===========================================
Files 783 783
Lines 60020 60021 +1
Branches 10161 10161
===========================================
- Hits 40468 40461 -7
- Misses 19549 19557 +8
Partials 3 3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
new_data['agreement_selected'] = { | ||
'annotators': {'all': True, 'ids': []}, | ||
'models': {'all': True, 'ids': []}, | ||
'ground_truth': False | ||
} |
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.
We don't care for existing configurations right? This isn't fully released yet, so I think its okay
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.
it was enabled for everyone for a few days - we had to turn it off after a few days - so the migration is needed
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.
It was left on for some people, so it definitely will need to migrate from old to new.
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.
@hakan458 can we check how many people have it configured? I feel that its okay to reset it them for them, but not certain.
This pull request improves the handling of the Agreement filter dropdown in the data manager table. The main focus is to ensure that the dropdown closes automatically after saving changes and to provide a way for the header cell to trigger closing the dropdown.
Dropdown behavior improvements:
onClose
callback toAgreementSelected.HeaderCell
, allowing the header cell to close the dropdown when needed. [1] [2]onSave
handler inTableHead.jsx
to automatically close the dropdown after saving agreement filters.