search

marigold-dev / batcher

The aim of the batch clearing dex is to enable users to deposit tokens with the aim of being swapped at a fair price with bounded slippage and almost no impermanent loss.
https://batcher.marigold.dev/
MIT License
10 stars 2 forks source link

Mainnet fix #397

Closed glottologist closed 1 year ago

glottologist commented 1 year ago

This PR contains the final fixed and checked contract for v 1.0.1 on mainnet.

Contract: The original audited contract was this commit (https://github.com/marigold-dev/batcher/commit/96c75a7bcb532eb34030d72aa447b4d60fd641bf). This fix should match that but with the addition of the fix to enforce the correct side plus the val fix for the more recent version of the ligo compiler.

Important review checks:

Contract is as expected per above Ghostnet is correct for ghostnet admin address (should be the Batcher testing address - tz1ca4batAsNxMYab3mUK5H4QRjY8drV4ViL) token metadata (incl. address and id etc) oracle details (address, asset name, decimals) Mainnet is correct for mainnet admin address (should be the current infra admin address) token metadata (incl. address and id etc) oracle details (address, asset name, decimals) The ghostnet contract has been deployed here: https://better-call.dev/ghostnet/KT1Gpu3ahjc9pyS4VrwMxsQ7KJWqrcqPoisR/operations

Once this has been reviewed I will deploy the mainnet contract and ask for a final review.

UI: The UI has additional fixes and features from the commit (as these were added after mainnet contract release)