-
Notifications
You must be signed in to change notification settings - Fork 499
Fix networkstate update behavior (DEV) #3767
Conversation
Solution: Just force a refresh on any state change. Also see: https://issuetracker.google.com/issues/144891976
@d4rken its much better than before. |
This was also working for me before.
This should be fixed by this PR, on which devices does it happen, and how did you test it? |
Tested on Pixel 3xl. There might be no 100%reliable solution... |
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.
Seems to be an improvement
Kudos, SonarCloud Quality Gate passed!聽 聽 |
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.
LGTM!
The network callback for onUnavailable() is not reliable.
Solution: Just force a refresh on any state change.
Also see:
https://issuetracker.google.com/issues/144891976
According to that ticket, it's fixed in a "future release", but on my Pixel 3a@Android11, it's not 馃し.
@BMItr noticed this during debugging.