-
Notifications
You must be signed in to change notification settings - Fork 27
fix(all) Progress the final sqlite message....can still take a while #1042
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
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #1042 +/- ##
==========================================
+ Coverage 41.06% 42.00% +0.93%
==========================================
Files 109 109
Lines 3090 3119 +29
Branches 295 295
==========================================
+ Hits 1269 1310 +41
+ Misses 1775 1766 -9
+ Partials 46 43 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
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.
not gross.
Adds finalization step for caching to show user more detail.
Fixes progress on transmissions.
Changes Sqlite saving to pre-report progress to allow detection for UI