Skip to content

fix: resolve QueryProvider and SocketProvider client/server boundary …#293

Open
Truphile wants to merge 2 commits into
StellarFlow-Network:mainfrom
Truphile:fix/query-socket-provider-client-boundary
Open

fix: resolve QueryProvider and SocketProvider client/server boundary …#293
Truphile wants to merge 2 commits into
StellarFlow-Network:mainfrom
Truphile:fix/query-socket-provider-client-boundary

Conversation

@Truphile
Copy link
Copy Markdown

@Truphile Truphile commented May 31, 2026

Closes #266


…issues

  • Add 'use client' directive to QueryProvider for proper client component handling
  • Wrap children in SocketProvider within RootLayout to ensure context availability
  • Add new useAssetPrice hook for granular price subscription
  • Refactor PriceFeedCard with memoized PriceValue component for performance
  • Update LivePrices component to use useAssetPrice hook
  • Fixes build errors: 'cannot serialize non-plain objects' and 'useSocketConnection must be used within SocketProvider'
  • Build now passes successfully with all routes generating correctly

Truphile and others added 2 commits May 31, 2026 01:38
…issues

- Add 'use client' directive to QueryProvider for proper client component handling
- Wrap children in SocketProvider within RootLayout to ensure context availability
- Add new useAssetPrice hook for granular price subscription
- Refactor PriceFeedCard with memoized PriceValue component for performance
- Update LivePrices component to use useAssetPrice hook
- Fixes build errors: 'cannot serialize non-plain objects' and 'useSocketConnection must be used within SocketProvider'
- Build now passes successfully with all routes generating correctly
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 31, 2026

@Truphile 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

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.

🧠 State-Isolation | Memoized Local Closures for Live Price Tickers

1 participant