fix(DockView): throw exception when dispose#8048
Conversation
|
🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The PR title and description suggest a functional fix in DockView, but the diff only shows a project file touch with no visible changes; please ensure the intended code changes are included or adjust the PR metadata accordingly.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The PR title and description suggest a functional fix in DockView, but the diff only shows a project file touch with no visible changes; please ensure the intended code changes are included or adjust the PR metadata accordingly.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Pull request overview
Updates the BootstrapBlazor.Server project dependency on BootstrapBlazor.DockView to pick up the upstream fix for the reported DockView dispose-time exception (issue #8047).
Changes:
- Bumped
BootstrapBlazor.DockViewNuGet package from10.0.13to10.0.14in the Server project.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8048 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 766 766
Lines 34157 34157
Branches 4697 4697
=========================================
Hits 34157 34157
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:
|
Link issues
fixes #8047
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Build: