Skip to content

fix: APP-582 User dropdown and "my orders" using Wallet Connect#2610

Merged
blushi merged 6 commits intodevfrom
fix-APP-582-wc-profile
Mar 3, 2025
Merged

fix: APP-582 User dropdown and "my orders" using Wallet Connect#2610
blushi merged 6 commits intodevfrom
fix-APP-582-wc-profile

Conversation

@blushi
Copy link
Member

@blushi blushi commented Feb 24, 2025

Description

https://regennetwork.atlassian.net/browse/APP-582


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • provided a link to the relevant issue or specification
  • provided instructions on how to test
  • reviewed "Files changed" and left comments if necessary
  • confirmed all CI checks have passed

How to test

  1. Connect with Wallet Connect from desktop or mobile
  2. Once connected, check out you can see your user profile image and name in the user dropdown
  3. Check out the tooltip text for "Edit profile" and "Settings" being disabled
  4. Go to "My orders" to see your crypto orders

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items
.

I have...

  • confirmed all author checklist items have been addressed
  • reviewed code correctness and readability
  • verified React components follow DRY principles
  • reviewed documentation is accurate
  • reviewed tests
  • manually tested (if applicable)

@blushi blushi marked this pull request as draft February 24, 2025 13:21
@netlify
Copy link

netlify bot commented Feb 24, 2025

Deploy Preview for regen-website ready!

Name Link
🔨 Latest commit a08b656
🔍 Latest deploy log https://app.netlify.com/sites/regen-website/deploys/67c57075c1d90d0008c4a886
😎 Deploy Preview https://deploy-preview-2610--regen-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Feb 24, 2025

Deploy Preview for terrasos ready!

Name Link
🔨 Latest commit a08b656
🔍 Latest deploy log https://app.netlify.com/sites/terrasos/deploys/67c57075678ec50008ea5383
😎 Deploy Preview https://deploy-preview-2610--terrasos.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@blushi blushi requested a review from r41ph February 25, 2025 10:15
@blushi blushi marked this pull request as ready for review February 25, 2025 10:15
@blushi
Copy link
Member Author

blushi commented Feb 25, 2025

@erikalogie @S4mmyb see testing instructions
although wallet connect cloud is having some maintenance at the moment so I could not add the deploy preview to the list of allowed URLs so you won't be able to connect, I'll post something here when it's done (should be around 4:30pm CET at the latest based on https://status.reown.com/incidents/xh858903ycn7)

@blushi blushi requested a review from r41ph February 25, 2025 14:01
Copy link
Contributor

@r41ph r41ph left a comment

Choose a reason for hiding this comment

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

@erikalogie
Copy link
Collaborator

@blushi looks like wallet connect is back, do you have do something now for this to be testable? I'm getting an "expired" error when scanning the desktop keplr mobile QR code with the mobile app.

@blushi
Copy link
Member Author

blushi commented Feb 25, 2025

@blushi looks like wallet connect is back, do you have do something now for this to be testable? I'm getting an "expired" error when scanning the desktop keplr mobile QR code with the mobile app.

I've added this deploy preview to the list of allowed URLs, so it's now testable (although I would recommend to wait a few minutes, sometimes it can take some time to be applied)
make sure to use a new QR code

@blushi blushi force-pushed the fix-APP-582-wc-profile branch from 85ee40b to a145575 Compare February 25, 2025 15:55
@erikalogie
Copy link
Collaborator

erikalogie commented Feb 25, 2025

I'm getting this warning even when logged in with Keplr extension and the credit card orders are displayed:

Screenshot 2025-02-25 at 1 51 32 PM

@blushi
Copy link
Member Author

blushi commented Feb 26, 2025

I'm getting this warning even when logged in with Keplr extension and the credit card orders are displayed:

Screenshot 2025-02-25 at 1 51 32 PM

Ok this happened after logging out from Wallet connect and then logging in with keplr extension straight after that, right? @erikalogie

@erikalogie
Copy link
Collaborator

I'm getting this warning even when logged in with Keplr extension and the credit card orders are displayed:
Screenshot 2025-02-25 at 1 51 32 PM

Ok this happened after logging out from Wallet connect and then logging in with keplr extension straight after that, right? @erikalogie

yes

@blushi
Copy link
Member Author

blushi commented Feb 27, 2025

I'm getting this warning even when logged in with Keplr extension and the credit card orders are displayed:
Screenshot 2025-02-25 at 1 51 32 PM

Ok this happened after logging out from Wallet connect and then logging in with keplr extension straight after that, right? @erikalogie

yes

This should be fixed now

@blushi blushi requested a review from r41ph February 27, 2025 13:27
@erikalogie
Copy link
Collaborator

LGTM

@blushi blushi force-pushed the fix-APP-582-wc-profile branch from f5cec1d to a08b656 Compare March 3, 2025 09:03
@blushi blushi merged commit a0c760f into dev Mar 3, 2025
16 of 18 checks passed
@blushi blushi deleted the fix-APP-582-wc-profile branch March 3, 2025 09:08
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.

3 participants