search

oasisprotocol / sapphire-paratime

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

Add deployment pattern recommendations #313

Closed aefhm closed 2 months ago

aefhm commented 3 months ago

Description

Add technical notes on proxies.

Relates to https://github.com/oasisprotocol/docs/pull/818.

netlify[bot] commented 3 months ago

Deploy Preview for oasisprotocol-sapphire-paratime canceled.

Name Link
Latest commit f7a4bf69d7841efd8f2a2a62ef2c9c83f74d3eb5
Latest deploy log https://app.netlify.com/sites/oasisprotocol-sapphire-paratime/deploys/66575b7983ecf10007e2b1bb
aefhm commented 2 months ago

Do you think we should rather rename the chapter to upgradable.md? I can't get rid of a feeling that the chapter is about setting up oracles or paymasters or proxy nodes at first glance when I see the "Proxies" title. The "Upgradable Contracts" is the title I think it rings the right bell for firstcomers.

I kind of want to 🤔. I hear you on the notion of proxy nodes. My concern with "Upgradeable Contracts" would be that the deterministic deploys are "Clones" so to speak with regards to OpenZeppelin terminology. Perhaps "Deployment Customization" or "Upgradable and Deterministic Contracts" can unify both sections? Or how about "Deployment Patterns?"