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

[dApp Notifications] Show successful dApp connection in Activity Center #20467

Open
ilmotta opened this issue Jun 14, 2024 · 10 comments · May be fixed by #20905
Open

[dApp Notifications] Show successful dApp connection in Activity Center #20467

ilmotta opened this issue Jun 14, 2024 · 10 comments · May be fixed by #20905

Comments

@ilmotta
Copy link
Contributor

ilmotta commented Jun 14, 2024

Feature

  • As a user
  • I want to see a new Activity Center Notification about my successful dApp connection
  • So that I can look them up later at a more appropriate time, and have a track record of my successful connections

Figma reference: https://www.figma.com/design/JZqS95JKcpZU13NHbJ9qVT/dApp-Interactions?node-id=47-30832&t=71hDvlz63qdheZ2z-0

Notes

  • Also requires changes in status-go.
  • As usual with all AC notifications, make sure the they're synchronized correctly across devices, and that the user can delete them and mark them as read/unread. In theory, these basic features will work out of the box, or with minimal changes.

Out of scope

Pressing on the notification to execute its default action will be a separate GH issue.

Resources

@ilmotta ilmotta added feature feature requests activity-center status-go wallet-connect: UI Adding missing UI on top of the existing functionality and polishing it based on the descopeddesings mobile-core labels Jun 14, 2024
@yqrashawn yqrashawn self-assigned this Jun 18, 2024
@yqrashawn
Copy link
Contributor

Stuck at the event below after scanning qr code at https://react-app.walletconnect.com/, no on-success or on-fail

(rf/reg-event-fx
 :wallet-connect/pair
 (fn [{:keys [db]} [url]]
   (let [web3-wallet (get db :wallet-connect/web3-wallet)]
     {:fx [[(log/spy :info :effects.wallet-connect/pair)
            {:web3-wallet web3-wallet
             :url         (log/spy :info url)
             :on-fail     #(log/error (log/spy :info "Failed to pair with dApp") {:error %})
             :on-success  #(log/info (log/spy :info "dApp paired successfully"))}]]})))

@yqrashawn
Copy link
Contributor

Network issue solved

@yqrashawn
Copy link
Contributor

Updates:

  • able to create the notification in status-go when dapp connected
  • able to get and show the notification in activity center

Next up:

  • Figma uses token context tag for dapp, but dapp log is not always a token logo
  • where to get dapp logo, do we have it persisted in mobile db already? How to identify dapp between mobile and go without relying on wallet connect?

wallet-connect proposer has dapp icons

    "proposer": {
      "publicKey": "...",
      "metadata": {
        "description": "React App for WalletConnect",
        "url": "https://react-app.walletconnect.com",
        "icons": [
          "https://avatars.githubusercontent.com/u/37784886"
        ],
        "name": "React App"
      }
    }

CleanShot 2024-06-28 at 14 44 09@2x

CleanShot 2024-06-28 at 14 47 03@2x

@clauxx
Copy link
Member

clauxx commented Jun 28, 2024

where to get dapp logo, do we have it persisted in mobile db already? How to identify dapp between mobile and go without relying on wallet connect?

We are persisting the dapps data (including the icon) when connecting in services/walletconnect/walletconnect.go.

Figma uses token context tag for dapp, but dapp log is not always a token logo

Not sure I follow. Is it that figma designs expect a token type and don't have a dapp type? If so, I guess we would need to add an extra type to the context tag (and update the design system as well).

@yqrashawn
Copy link
Contributor

Thanks for the info @clauxx

I posted some questions about wakuext_addWalletConnectSession and wallet_addWalletConnectSession at https://discord.com/channels/1210237582470807632/1217175655729467492/1257161311566168136

@yqrashawn
Copy link
Contributor

updates:
working on other issue, will come back to this once mobile migrated to wallet connect v2 api

@clauxx
Copy link
Member

clauxx commented Jul 8, 2024

Blocked till #20637 is merged

@ilmotta ilmotta added this to the 2.31.0 Beta milestone Jul 15, 2024
@ilmotta
Copy link
Contributor Author

ilmotta commented Jul 15, 2024

FYI @yqrashawn, we are postponing this issue for 2.31 as we believe it's risky to push it for 2.30.

@clauxx clauxx added dapp-interactions and removed wallet-connect: UI Adding missing UI on top of the existing functionality and polishing it based on the descopeddesings labels Jul 16, 2024
@yqrashawn
Copy link
Contributor

yqrashawn commented Aug 6, 2024

next up:
when ready
merge status-go PR, push new tag, update status-go-version.json and merge mobile PR

@churik
Copy link
Member

churik commented Sep 19, 2024

defer due to #20905 (comment)

@churik churik removed this from the 2.31.0 Beta milestone Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: QA
Development

Successfully merging a pull request may close this issue.

5 participants