Skip to content

Commit c5c6f9d

Browse files
committed
[TOOL-2999] SDK: automatically sign message for SIWE in connect UI for Ecosystem wallets (#5914)
--- title: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" --- If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on improving the `SignatureScreen` component by updating the signing flow for different wallet types and enhancing the user experience by removing unnecessary prompts for signing messages. ### Detailed summary - Added `isHeadlessSignSupported` function to determine wallet support. - Updated `SignatureScreen` logic to handle headless signing for supported wallets. - Refactored tests to improve clarity and organization. - Enhanced error handling and loading states in the UI. - Introduced mocks for `useActiveWallet` and `useAdminWallet` in tests. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
1 parent b14aa54 commit c5c6f9d

File tree

3 files changed

+238
-173
lines changed

3 files changed

+238
-173
lines changed

.changeset/six-snails-reflect.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"thirdweb": patch
3+
---
4+
5+
Do not prompt user for signing message for SIWE auth in Connect UI for Ecosystem wallets

0 commit comments

Comments
 (0)