Skip to content

Fix several app non responsive issues#8461

Merged
BillCarsonFr merged 1 commit into
developfrom
feature/bca/fix_several_anr
May 23, 2023
Merged

Fix several app non responsive issues#8461
BillCarsonFr merged 1 commit into
developfrom
feature/bca/fix_several_anr

Conversation

@BillCarsonFr
Copy link
Copy Markdown
Member

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other :

Content

Fixes several ANR reported in sentry
Fixes #8454
Fixes #8455

This is a quick fix, but it would be a good idea to have all Task<..> not dispatching on Main by default.

Motivation and context

Screenshots / GIFs

Tests

  • Step 1
  • Step 2
  • Step ...

Tested devices

  • Physical
  • Emulator
  • OS version(s):

Checklist

@BillCarsonFr BillCarsonFr requested a review from bmarty May 23, 2023 06:04
@BillCarsonFr BillCarsonFr force-pushed the feature/bca/fix_several_anr branch from fd8d2e9 to 2ea6cdb Compare May 23, 2023 06:31
Copy link
Copy Markdown
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@BillCarsonFr BillCarsonFr merged commit 591b08f into develop May 23, 2023
@BillCarsonFr BillCarsonFr deleted the feature/bca/fix_several_anr branch May 23, 2023 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ANR in handleDeleteLocalRooms ANR in NotificationState in updateQueuedEvents

2 participants