-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Open
Description
Followup things to be done after merging #4549:
- Voyager needs to check whether a token exists with the given salt before deploying it. If it exists, it should use that type name.ZKGM must check the decimals of the previously registered token on recv and also should change the name, symbol and the description accordingly.ZKGM must save the token's typename under a certain quote token so that we don't have multiple tokens for the same quote token.Add
PacketByHash
impl.AddIbcInterface::IBC_MOVE_SUI
Makeclient_info
queryable in the voyager sui state module (for state lens stuff)Read the objects directly instead of dry runningImpl fetch blocks s.t. it can catch up. We probably don't want to read all blocks but instead, use a subscription mechanism.Gas simulation or configurable gas.Hussein's commentWe miss a verification on scalarApi.AssertIsEqual(&c.InnerWitness.Public[1], commitmentHash) And an extra assertion eq(len(innerwitness.public), 2)
.To pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
Activity
Herdethegbar commentedon Jul 8, 2025
Are they adding sui, or are you just guessing?
aeryz commentedon Jul 9, 2025
We are working on it :)