Add ERC: Consolidated Dynamic Storage (CDS) #774
Conversation
Committing updated header Co-authored-by: Andrew B Coathup <[email protected]>
wisecameron
changed the title
eip-review-bot
changed the title
…ERCs into wisecameron/erc-cds
… architecture diagram)
SamWilsn
left a comment
SamWilsn
left a comment
There was a problem hiding this comment.
Few additional comments while you're working on this.
…umentation. Enhanced clarity on the capabilities and benefits of Consolidated Dynamic Storage, emphasizing its role as a comprehensive storage abstraction layer. Updated permission management table for improved structure and clarity. Changed SPDX license identifier in the CDSMinimal.sol file to CC0-1.0.
…ved flowery language. Focused on communicating key points with more clarity.
…clarity and conciseness. Emphasize the role of CDS as a foundational storage primitive and its benefits in upgradeability and decentralization.
…ndancy and enhancing clarity and formatting.
Co-authored-by: Sam Wilson <[email protected]>
|
I have substantially improved the majority of the proposal - I also have a completely refactored rationale section in the works here. I really appreciate that you guys have let me put this on the backburner for some time -- I needed to step away for a while due to work constraints and just to cool off and let these ideas settle into a more cohesive whole. |
…azing feedback :)
…ERCs into wisecameron/erc-cds
…vation section, improved interface clarity.
… creation and extension of mapped structs and storage spaces.
|
The commit f2f6e8e (as a parent of ce2bdae) contains errors. |
SamWilsn
left a comment
SamWilsn
left a comment
There was a problem hiding this comment.
Looking good! Still need to remove the bold/italics from the UPPERCASE keywords, and move example code into a reference implementation section.
Hey thank you man! Update: |
5 participants
Reference implementation:
https://github.com/wisecameron/ConsolidatedDynamicStorage/blob/main/contracts/CDSMinimal.sol