-
Notifications
You must be signed in to change notification settings - Fork 544
[TOOL-2815] Nebula - handle disconnected wallet state, use connected account address in execute config #5821
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5821 +/- ##
=======================================
Coverage 54.11% 54.11%
=======================================
Files 1102 1102
Lines 59202 59202
Branches 4885 4885
=======================================
Hits 32037 32037
Misses 26441 26441
Partials 724 724
*This pull request uses carry forward flags. Click here to find out more. |
Merge activity
|
…account address in execute config (#5821) ## Problem solved Short description of the bug fixed or feature added <!-- start pr-codex --> --- ## PR-Codex overview This PR primarily focuses on refining the chat functionality and user experience in the `ChatBar` and `ChatPageContent` components. It enhances wallet connection handling, streamlines message sending, and introduces a dialog for when the wallet is disconnected. ### Detailed summary - Added `e.preventDefault()` in `ChatBar` to prevent default behavior on Enter key. - Removed unused wallet address fetching in several components. - Updated `config` type to allow `null` in `Chat` API. - Introduced `WalletDisconnectedDialog` in `ChatPageContent`. - Enhanced state management for showing different login pages. - Improved context filter updates based on wallet address and active chain. - Added conditional rendering for wallet connection prompts. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
36ae01a
to
1e27491
Compare
Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR focuses on enhancing user experience in the chat application by preventing default behavior on the Enter key, simplifying authentication logic, and introducing a modal dialog for wallet connection issues.
Detailed summary
Enter
key inChatBar
.getAuthTokenWalletAddress
calls for account address retrieval.promptNebula
to handleconfig
as nullable.showConnectModal
state for wallet connection prompts inChatPageContent
.WalletDisconnectedDialog
for handling wallet disconnection scenarios.