-
Notifications
You must be signed in to change notification settings - Fork 16
feat: add UPS power #1874
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
feat: add UPS power #1874
Conversation
WalkthroughAdded two new UPS power fields (nominalPower and currentPower) to the GraphQL model, extended parsed UPS data with NOMPOWER, and updated the resolver to parse nominal power and compute current power from nominalPower and loadPercentage. Changes
Sequence Diagram(s)(omitted) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1874 +/- ##
==========================================
+ Coverage 46.40% 46.43% +0.02%
==========================================
Files 954 954
Lines 59791 59810 +19
Branches 5538 5547 +9
==========================================
+ Hits 27749 27772 +23
+ Misses 31923 31919 -4
Partials 119 119 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
This plugin has been deployed to Cloudflare R2 and is available for testing. |
Summary by CodeRabbit
New Features
Tests
✏️ Tip: You can customize this high-level summary in your review settings.