-
Notifications
You must be signed in to change notification settings - Fork 3.2k
fix: BROS-486: Fix hotkeys initialization #8909
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
Conversation
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
✅ 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-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 #8909 +/- ##
===========================================
- Coverage 66.52% 57.73% -8.80%
===========================================
Files 828 561 -267
Lines 64405 40443 -23962
Branches 10937 10938 +1
===========================================
- Hits 42847 23350 -19497
+ Misses 21554 17089 -4465
Partials 4 4
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:
|
|
/git merge
|
|
/git merge
|
This PR addresses the issue when the
enableHotkeyssettings was not considered during LSF initialization. Now it's working the same way as inSettingsStore#toggleHotkeys().This is necessary for "Labeling Hotkeys" setting to properly work.