search

oasisprotocol / sapphire-paratime

The Sapphire ParaTime monorepo.
https://oasisprotocol.org/sapphire
Apache License 2.0
34 stars 24 forks source link

Update documented addresses #299

Closed aefhm closed 3 months ago

aefhm commented 4 months ago

Description

Add Celer IM message executor. Add Safe Singleton Factory.

TODO

netlify[bot] commented 4 months ago

Deploy Preview for oasisprotocol-sapphire-paratime ready!

Name Link
Latest commit 5d2099d78527964e84d4c720f87000bc8a4788aa
Latest deploy log https://app.netlify.com/sites/oasisprotocol-sapphire-paratime/deploys/664132f25563d40008f3e169
Deploy Preview https://deploy-preview-299--oasisprotocol-sapphire-paratime.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

aefhm commented 3 months ago

Pulling back from history our deterministic proxy https://github.com/safe-global/safe-singleton-factory/pull/78

aefhm commented 3 months ago

@matevz one more look? I think changes requested are there now.

aefhm commented 3 months ago

Everything seems correct

The 'Contract Addresses and Deployments' header does seem a bit long, but that's not a blocker.

Oh yeah. Hmm, maybe we should just have 2 total sections on this page "Addresses" with two sections 1 for "Contracts" and another for the bridged tokens instead.

CedarMist commented 3 months ago

Am happy to merge and then look at the docs from a higher-level perspective afterwards, having all the standard addresses on one page is good and easy to read.