Skip to content
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

[INV-3715] Implement Heartbeat monitoring for network connectivity #3727

Merged
merged 11 commits into from
Dec 2, 2024

Conversation

LocalNewsTV
Copy link
Collaborator

@LocalNewsTV LocalNewsTV commented Dec 2, 2024

Overview

This PR includes the following proposed change(s):

Handling These Network Scenarios

  • Initial Network State
  • Manual User Disconnect/Reconnect
  • Rolling API checks to maintain current network status
  • Automatic re-attempts to connect if disconnected

All changes affect Mobile Only

  • Closes Utilize Health Check to determine if Online/Offline #3715
  • If User determined offline, communicate to user, then attempt to reconnect in background.
  • Introduce two flag approach to connected logic. Based on User Consent and Network Availability
    • Ping network in intervals to determine network availability
      • Increase ping frequency with subsequent failures to avoid spamming to the void
    • Enable/Disable Pings based on User consent (administrative status)
  • Persist NetworkState values in lieu of pinging immediately on load.

Copy link

sonarqubecloud bot commented Dec 2, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@LocalNewsTV LocalNewsTV merged commit d29e3a4 into dev Dec 2, 2024
6 of 10 checks passed
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.

Utilize Health Check to determine if Online/Offline
2 participants