Agoric / agoric-sdk

monorepo for the Agoric Javascript smart contract platform
Apache License 2.0
327 stars 208 forks source link

PFM transfer covering remote chain A's view of remote B's denoms #10006

Open turadg opened 2 months ago

turadg commented 2 months ago

What is the Problem Being Solved?

Per @0xpatrickdev , we need and don't have any test scenarios for building PFM messages which require us to know RemoteChainA's view of RemoteChainB's denoms.

Description of the Design

https://github.com/Agoric/agoric-sdk/pull/9491 made progress in pursuit of,

However this is smaller in scope. The scope of this is testing PFM and the pfm memo. (Perhaps through sendAnywhere contract)

Additional discussion

### Tasks
- [ ] https://github.com/Agoric/agoric-sdk/pull/10329
- [ ] implementation and test for COA
- [ ] https://github.com/Agoric/agoric-sdk/issues/10445

Security Considerations

Scaling Considerations

Test Plan

Upgrade Considerations

dckc commented 2 months ago

A relevant detail: "chain A's view of remote B's denoms" is supposed to be called getLocalDenom as in chainA.getLocalDenom(chainBdenom), according to July 31 discussion https://github.com/Agoric/agoric-sdk/pull/9814#discussion_r1699134254 where we dug up earlier API drafts.

A WIP version is included in