snaps icon indicating copy to clipboard operation
snaps copied to clipboard

Amon Hen V2 Snaps API Plan

Open ajeetd opened this issue 1 year ago • 3 comments

This is a research and planning story to consider an approach to handling the points discussed in this conversation: https://consensys.slack.com/archives/C03Q3TCUVRT/p1720794034563159

Specifically:

  • Currently Snaps are set to the current globally selected chain and this works now because Snaps are always set to the same chain as whatever the user sees in the UI
  • Going forward this will no longer be the case. We will start to see confirmations for a different chain than the globally selected chain. This means for instance that transaction insight snaps wont have the correct chain state.

Suggestions offered in the conversation:

  • Tie selected network for all Snaps to whatever network is represented in UI. If we are on a confirmation, all Snaps could have teh same network selected as the dapp that is requesting the confirmation. Otherwise use the globally selected network.
  • Set the selected chain of each tx insight snap to whatever chain is relevant for the current confirmation if there are any possible insights for this confirmation

See linked thread for the whole conversation.

Acceptance Criteria

  • At this point, put together a one pager proposal of an approach.

ajeetd avatar Jul 15 '24 13:07 ajeetd

@Mrtenz Please Slack thread linked above for context. No action required at this time other than being aware of the context and starting to consider approaches when time available.

ajeetd avatar Jul 16 '24 12:07 ajeetd

@Mrtenz - please close unless there's anything else left here.

ajeetd avatar Aug 30 '24 13:08 ajeetd

This work isn't merged yet so I think we should keep this open for now.

Mrtenz avatar Aug 30 '24 13:08 Mrtenz