-
Notifications
You must be signed in to change notification settings - Fork 13k
fix: status-text update in real time #35378
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
fix: status-text update in real time #35378
Conversation
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
🦋 Changeset detectedLatest commit: 76a149b The changes in this PR will be included in the next version bump. This PR includes changesets to release 36 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
bd8efd8 to
d3d70be
Compare
|
Hi @juliajforesti and @reetp , I appreciate the work you and the team do in maintaining this project. This PR has been open for a while, and I’d love to get your feedback whenever you have time. Please let me know if there’s anything I can improve. Thank you! |
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
…d then later just the status + statusDefault)
Kody Review CompleteGreat news! 🎉 Keep up the excellent work! 🚀 Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
…s and prevent redundant updates
Kody Review CompleteGreat news! 🎉 Keep up the excellent work! 🚀 Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #35378 +/- ##
===========================================
- Coverage 61.15% 56.37% -4.78%
===========================================
Files 3009 2583 -426
Lines 71554 64729 -6825
Branches 16378 14734 -1644
===========================================
- Hits 43758 36493 -7265
- Misses 24830 25915 +1085
+ Partials 2966 2321 -645
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Hi there, thanks for the contribution! 🚀 💯 Closing this PR because it's stale now and it was already fixed here. Questions? Help needed? Feature Requests?
|
Screen Recording 2025-04-26 at 2.09.23 PM.mp4
Proposed changes (including videos or screenshots)
Screen.Recording.2025-04-26.at.2.09.23.PM.mp4
Issue(s)
Closes #35294
Steps to test or reproduce
Further comments
I have refactor the code please do let me know know if need other changes required