Skip to content
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

Handle add/switch account flow on WC message signing method #5282

Merged

Conversation

eniolam1000752
Copy link
Contributor

@eniolam1000752 eniolam1000752 commented Aug 31, 2023

What was the problem?

This PR resolves #5278

How was it solved?

  • Added RequstSummary component to the RequestSignMessageDialog multi step
  • Refactor RequestSummary to work properly in message mode
  • Fix failing unit tests

How was it tested?

  • Request message signing via wallet connect from an external dApp.
  • Test for scenarios if signing account is not available in the user's account list.
  • Test for scenarios if signing account is not the current account.
  • Try to sign a wallet connect request message using the hardware wallet

@eniolam1000752 eniolam1000752 marked this pull request as ready for review August 31, 2023 14:21
ManuGowda
ManuGowda previously approved these changes Sep 1, 2023
@@ -119,7 +119,7 @@ export const useSession = () => {
);

useEffect(() => {
if (signClient?.session && !hasLoaded) {
if (signClient?.session && !hasLoaded && isEnabled) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldnt we loadSessions if isEnabled become enabled? Dont we need it in the dep array in that case?

oskarleonard
oskarleonard previously approved these changes Sep 1, 2023
@ManuGowda ManuGowda merged commit 133a692 into release/3.0.0 Sep 1, 2023
6 checks passed
@ManuGowda ManuGowda deleted the 5278-implement-wc-message-signing-account-journey branch September 1, 2023 13:10
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.

None yet

3 participants