Skip to content

Conversation

@Heliophist
Copy link
Contributor

Hopefully fixed

@github-actions
Copy link

github-actions bot commented Sep 28, 2025

🚀 Expo preview is ready!

  • Project → audacity-music-club
  • Platforms → android, ios
  • Runtime Version → 1.0.0
  • More info

Learn more about 𝝠 Expo Github Action

@Heliophist Heliophist marked this pull request as ready for review November 9, 2025 19:12
Copilot AI review requested due to automatic review settings November 9, 2025 19:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds internet connectivity monitoring and a visual notification banner to alert users when they're offline.

  • Added @react-native-community/netinfo dependency and NoInternetBanner component
  • Updated Expo SDK from v51 to v54
  • Integrated the banner into the main App component

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/components/NoInternetBanner.js New component that monitors network state and displays an animated banner when offline
package.json Added netinfo dependency and upgraded Expo to v54
App.js Wrapped NavigationContainer in a View and added NoInternetBanner overlay

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Heliophist Heliophist requested a review from Pramad712 November 9, 2025 19:27
Copy link
Contributor

@Pramad712 Pramad712 left a comment

Choose a reason for hiding this comment

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

👍

@Pramad712 Pramad712 merged commit 9a2d57b into main Nov 10, 2025
5 checks passed
@Pramad712 Pramad712 deleted the no-wifi-banner-new branch November 10, 2025 00:27
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.

2 participants