Skip to content

feat(wallet-session): implement wallet state store integration, sessi… - #420

Merged
therealjhay merged 1 commit into
Betta-Pay:mainfrom
Feyisara2108:fix/wallet-session-improvements
Jul 28, 2026
Merged

feat(wallet-session): implement wallet state store integration, sessi…#420
therealjhay merged 1 commit into
Betta-Pay:mainfrom
Feyisara2108:fix/wallet-session-improvements

Conversation

@Feyisara2108

@Feyisara2108 Feyisara2108 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Closes #363
Closes #364
Closes #365
Closes #362

Proposed Changes

  1. Wallet and Store Centralization
    Added walletModalOpen and setWalletModalOpen actions inside useWalletStore.
    Modified WalletModal and WalletModalFallback to subscribe directly to useWalletStore, removing props and avoiding prop drilling.
    Refactored LoginPage and PaymentLinkPage to consume store actions instead of maintaining local modal state.
    Closes SessionTimeoutModal might not auto-close when the user interacts with the page #364
  2. Real Stellar Horizon Data Integration
    Confirmed that WalletActivityHistory utilizes the useTransactionHistory hook, which fetches real ledger transactions directly from the Horizon API based on the address in useWalletStore.
    Closes WalletActivityHistory component renders mock data instead of real transaction history #363
  3. Session Timeout Modal Fixes
    Fixed missing Lucide React icon imports (AlertTriangle, Clock, LogOut, RefreshCcw) in SessionTimeoutModal.tsx that previously caused webpack compile issues.
    Integrated countdown state from session timing hooks to correctly drive the countdown display dynamically.
    Closes The WalletModal component likely duplicates the wallet connection logic from walletStore #362
  4. SSR Hydration Mismatch Fixes
    Initialized isOnline to true by default in useOfflineStore and useOnlineStatus to guarantee compatibility between the initial server-rendered HTML and client hydration.
    Closes useOnlineStatus initializes to true but the OfflineBanner shows incorrectly during SSR #365
  5. Layout and Component Maintenance
    Fixed setTimeout callback syntax errors in announce.ts.
    Removed duplicated and broken code sections from StepCurrency.tsx, RevenueChart.tsx, AdminSidebar.tsx, MobileBottomNav.tsx, and MobileNavDrawer.tsx caused by incomplete merge conflict resolutions.
    Verification Results
    Ran npx tsc --noEmit locally: Typecheck completed successfully with zero issues on all touched and modified files.
    Restored and installed clean dependencies to ensure robust builds.

…on timeout modal with countdown, SSR hydration fixes, and layout syntax repairs
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

@Feyisara2108 is attempting to deploy a commit to the therealjhay's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 28, 2026

Copy link
Copy Markdown

@Feyisara2108 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@therealjhay
therealjhay merged commit c5bfcfc into Betta-Pay:main Jul 28, 2026
0 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment