Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

dev: Remove pallet::without_storage_info 🦀 #93

Closed
EvolveArt opened this issue Mar 26, 2023 · 0 comments
Closed

dev: Remove pallet::without_storage_info 🦀 #93

EvolveArt opened this issue Mar 26, 2023 · 0 comments
Labels
enhancement New feature or request
Milestone
Execute Starknet ...

Comments

@EvolveArt
Copy link
Collaborator

The ContractClassWrapper uses a BTreeMap to store entrypoints.

We should switch to a [BoundedBTreeMap](https://crates.parity.io/frame_support/storage/bounded_btree_map/index.html) as recommended by the substrate repo here.

On a general note, we should clearly state in some docs the boundaries for bounded values e.g MaxProgramSize, MaxCalldataSize etc
@EvolveArt EvolveArt added the enhancement New feature or request label Mar 26, 2023
@EvolveArt EvolveArt mentioned this issue Mar 26, 2023
Merged
10 tasks
@AbdelStark AbdelStark moved this to 🔖 Ready in Madara Mar 27, 2023
@EvolveArt EvolveArt mentioned this issue Mar 29, 2023
Merged
8 tasks
@github-project-automation github-project-automation bot moved this from 🔖 Ready to ✅ Done in Madara Mar 29, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Apr 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
Madara
Status: ✅ Done
Milestone
Execute Starknet contracts.
Development

No branches or pull requests
2 participants
@EvolveArt @AbdelStark