-
Notifications
You must be signed in to change notification settings - Fork 109
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
Timeline : remove some code from main thread #2841
Conversation
Quality Gate passedIssues Measures |
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2841 +/- ##
========================================
Coverage 73.95% 73.95%
========================================
Files 1531 1531
Lines 36505 36505
Branches 7069 7069
========================================
Hits 26999 26999
Misses 5804 5804
Partials 3702 3702 ☔ View full report in Codecov by Sentry. |
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.
Thanks for you investigation and for ensuring that the app is always as snappy as ever!
Timber.d("Can't paginate $direction on room ${matrixRoom.roomId} with paginationStatus: ${backPaginationStatus.value}") | ||
} else { | ||
Timber.e(error, "Error paginating $direction on room ${matrixRoom.roomId}") | ||
withContext(dispatcher) { |
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.
Can be written like this: withContext(NonCancellable + dispatcher) {
I think, but not a big deal as using NonCancellable
is a temporary workaround.
Just small clean up of code launched on main thread.