Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spec update for unlimited assets #62

Merged

Conversation

algorandskiy
Copy link
Contributor

@algorandskiy algorandskiy commented Feb 18, 2022

Allow unlimited assets and apps.
Changes:

  • Remove max assets parameter
  • State apps/assets are unlimited
  • Remove maximum minimum balance
  • Modify storage in maps

@algorandskiy algorandskiy force-pushed the pavel/unlimited-assets branch from 82052bb to a01a625 Compare February 18, 2022 20:29
dev/ledger.md Outdated Show resolved Hide resolved
dev/ledger.md Outdated Show resolved Hide resolved
dev/ledger.md Outdated Show resolved Hide resolved
dev/ledger.md Outdated Show resolved Hide resolved
Copy link
Contributor

@tsachiherman tsachiherman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

few small requests, looks good otherwise.

@algorandskiy
Copy link
Contributor Author

Fixed CR comments

@algofoundation algofoundation merged commit d5ac876 into algorandfoundation:master Feb 24, 2022
tsachiherman added a commit to algorand/go-algorand that referenced this pull request Mar 2, 2022
## Summary

This PR adds support of for unlimited assets.

Done:
- [x] Update this title
- [x] Complete & Merge the implementation of the accounts prefetched during the validation (#3666)
- [x] Create & Merge a spec update ( algorandfoundation/specs#62 (comment) )
- [x] Create & Merge a consensus upgrade (#3674)

## Test Plan

Extend, update and add new unit tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants