Skip to content

[Enhance] show VPN connection state when failing to connect to electrum server#590

Open
doey-noncelab wants to merge 5 commits intodevelopfrom
enhance/vpn-status
Open

[Enhance] show VPN connection state when failing to connect to electrum server#590
doey-noncelab wants to merge 5 commits intodevelopfrom
enhance/vpn-status

Conversation

@doey-noncelab
Copy link
Collaborator

@doey-noncelab doey-noncelab commented Feb 5, 2026

수정 사항

1. Added

  • wallet_home AppBar 관련
    • 일렉트럼 서버 에러 발생 시, VPN이 켜져 있으면 에러 메시지 표시
    • 에러 메시지 locale 문구 추가
  • NetworkStatus
    • vpnBlocked enum 멤버 추가

2. Refactor

[AS-IS]

  • AppGuard의 역할 (메인: 앱 생명주기 관리)
    • Connectivity() 객체 생성
    • onConnectivityChanged 구독
    • ConnectivityProvider.setIsInternetOn() 호출
  • ConnectivityProvider: 이름과 달리 단순 상태 저장

[TO-BE]

  • AppGuard: ConnectivityProvider 참조 (네트워크 관련 로직 없음, ConnectivityProvider의 메서들을 직접 호출하지 않음)
  • ConnectivityProvider
    • Connectivity() 객체 생성
    • onConnectivityChanged 구독
    • onVpnStatusChanged 구독
    • 네트워크 상태 관리 및 제공 (필드 리네임: isNetworkOn ➔ isInternetOn, 추가: isVpnActive)

Expected behavior

Case Internet VPN 화면 표시
1 ON OFF 일렉트럼 서버 에러 시 "일렉트럼 서버 오류" 표시
2 ON ON 일렉트럼 서버 에러 시, "VPN 연결" 표시
3 OFF OFF "네트워크 끊김"
4 OFF ON 일렉트럼 서버 에러 시, VPN이 라우팅 불가 상태이므로 "네트워크 끊김"

fix(tx-analysis-widget): fix UI; fix custom range not applying correctly
refactor(connectivity-provider): move logic from AppGuard to ConnectivityProvider; distinguish internet vs vpn
refactor(connectivity-provider): move logic from AppGuard to ConnectivityProvider; distinguish internet vs vpn
@doey-noncelab doey-noncelab added ♻️ refactor 리팩토링 이슈 ✨enhance UI 및 사용성 개선 이슈 labels Feb 5, 2026
@doey-noncelab doey-noncelab added this to the Deploy 0.8.0/3.7.0 milestone Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨enhance UI 및 사용성 개선 이슈 ♻️ refactor 리팩토링 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants