Explore-Reown-For-WalletConnect#2156

Merged

OmarHatem28 merged 18 commits intomainfrom

Explore-Reown-For-WalletConnect

Apr 24, 2025

Contributor

Blazebrain commented Apr 3, 2025 •

edited

Loading

Setup for Solana Chain
Setup for EVM chain
Enable signing messages and transactions on Solana Chain
Fix issues related to signinig of transactions on Solana chain
Enable signing messages and transactions on EVM Chains
Fix issues related to signinig of transactions on EVM chains
Decode message to human readable format
Add readable text for signedTypedDatav4 and sendTransaction event requests
Modify styling for bottomsheetModel
Switch minimum deployment target to 13.0
Gracefully handle errors
Fix issues discovered while testing across dApps
Remove unneeded code and files
Rename files to match classes and vice versa
Add maven jitpack repository for reown.com/yttrium repo
Add better folder structuring to entire feature
Add localization for all feature related texts
Add testnet and devnet access for Solana Chain Service
Add a better error message for Moralis NFT fetch

Issue Number (if Applicable): Fixes #

Description

Please include a summary of the changes and which issue is fixed / feature is added.

Pull Request - Checklist

Initial Manual Tests Passed
Double check modified code and verify it with the feature/task requirements
Format code
Look for code duplication
Clear naming for variables and methods

Blazebrain added 3 commits

April 2, 2025 14:08


          Reown WalletConnect Implementation

d3ba4d1

- Setup for Solana Chain
- Setup for EVM chain
- Enable signing messages and transactions on Solana Chain
- Fix issues related to signinig of transactions on Solana chain
- Decode mesage to human readable format - WIP


          Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

2f4873d

…Explore-Reown-For-WalletConnect


          Reown WalletConnect Implementation Continuation:

d7a7982

- Add readable text for signedTypedDatav4 and sendTransaction event requests
- Modify styling for bottomsheetModel
- Switch minimum deployment target to 13.0
- Gracefully handle errors
- Fix issues discovered while testing across dApps

Blazebrain self-assigned this

Blazebrain requested a review from OmarHatem28

April 3, 2025 21:16

Blazebrain added 6 commits

April 4, 2025 04:33


          Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

a2b596f

…Explore-Reown-For-WalletConnect


          chore: ReownWalletConnectImplementation (Cont.)

d6918ab

- Remove unneeded code and files
- Rename files to match classes and vice versa
- Add maven jitpack repository for reown.com/yttrium repo
- Add better folder structuring to entire feature
- Add localization for all feature related texts
- Add testnet and devnet access for Solana Chain Service
- Add a better error message for Moralis NFT fetch


          Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

7bba052

…Explore-Reown-For-WalletConnect


          feat:

c38c521

- Add field to enter wc uri directly
- Handle bottomsheet not coming up after initial display
- Remove unneeded print statements
- Remove double display of success popup for session proposal bottomsheets


          fix:

8f43c18

- Add visual feedback to WC RelayClient Errors
- Adjust theme for bottomSheets


          Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

e084442

…Explore-Reown-For-WalletConnect

OmarHatem28 linked an issue

that may be closed by this pull request

solana_signTransaction (WalletConnect) #2185

Closed

OmarHatem28 mentioned this pull request

solana_signTransaction (WalletConnect) #2185

Closed

Blazebrain and others added 9 commits

April 14, 2025 09:08


          Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

61f4029

…Explore-Reown-For-WalletConnect


          Testing a theroy

168a9d3


          Temporarily add flags back

26f66b8


          Remove added flags and update proguard rules

c4ee624


          fix(android): update proguard-rules.pro and drop minify flags (#2199)

92491de

Co-authored-by: Omar Hatem <omarh.ismail1@gmail.com>


          Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

57356f8

…Explore-Reown-For-WalletConnect


          Merge branch 'Explore-Reown-For-WalletConnect' of https://github.com/…

cd12d2a

…cake-tech/cake_wallet into Explore-Reown-For-WalletConnect


          Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

a30f40c

…Explore-Reown-For-WalletConnect


          fix: Switch bottomsheet result from signOne to signAll

575008c

Blazebrain marked this pull request as ready for review

April 24, 2025 14:46

OmarHatem28 approved these changes

View reviewed changes

lib/src/screens/wallet_connect/widgets/wc_session_auth_request_widget.dart Outdated Show resolved Hide resolved

OmarHatem28 merged commit e6c9cf5 into main

4 checks passed

OmarHatem28 deleted the Explore-Reown-For-WalletConnect branch

April 24, 2025 16:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet