Skip to content

Feature/fix wallet context#276

Closed
AugistineCreates wants to merge 10 commits into
NOVUS-X:mainfrom
AugistineCreates:feature/fix-wallet-context
Closed

Feature/fix wallet context#276
AugistineCreates wants to merge 10 commits into
NOVUS-X:mainfrom
AugistineCreates:feature/fix-wallet-context

Conversation

@AugistineCreates
Copy link
Copy Markdown
Contributor

This pr closes #233

Resolved performance issues in WalletContext caused by repeated initialization and uncleaned event listeners. This improves stability and prevents memory leaks during prolonged usage.

Changes made
-Refactored useEffect hooks to avoid unnecessary re-runs and repeated wallet setup
-Added proper cleanup functions to remove wallet kit event listeners on unmount
-Memoized the context provider value using useMemo to prevent avoidable re-renders

Result

-Eliminates memory leaks from lingering listeners
-Reduces unnecessary re-renders across components using the context
-Improves overall performance and responsiveness of wallet interactions

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Apr 25, 2026

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

@AugistineCreates AugistineCreates deleted the feature/fix-wallet-context branch April 28, 2026 01:30
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.

Frontend: Fix Memory Leaks in WalletContext

1 participant