feat(wallet-session): implement wallet state store integration, sessi… - #420
Merged
therealjhay merged 1 commit intoJul 28, 2026
Merged
Conversation
…on timeout modal with countdown, SSR hydration fixes, and layout syntax repairs
|
@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. |
|
@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! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #363
Closes #364
Closes #365
Closes #362
Proposed Changes
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
SessionTimeoutModalmight not auto-close when the user interacts with the page #364Confirmed that WalletActivityHistory utilizes the useTransactionHistory hook, which fetches real ledger transactions directly from the Horizon API based on the address in useWalletStore.
Closes
WalletActivityHistorycomponent renders mock data instead of real transaction history #363Fixed 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
WalletModalcomponent likely duplicates the wallet connection logic fromwalletStore#362Initialized isOnline to true by default in useOfflineStore and useOnlineStatus to guarantee compatibility between the initial server-rendered HTML and client hydration.
Closes
useOnlineStatusinitializes totruebut theOfflineBannershows incorrectly during SSR #365Fixed 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.