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

Change numbers to ids #278

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Change numbers to ids #278

wants to merge 18 commits into from

Conversation

Neeharika-Sompalli
Copy link
Member

Change all numbers to IDs

Neeharika-Sompalli and others added 16 commits April 12, 2023 18:15
* Add HCS state proto

Signed-off-by: Michael Tinker <[email protected]>

* Add running hash, delete EntityNumber

Signed-off-by: Michael Tinker <[email protected]>

* add account and token

* add account and token

* add token

* add tokenRelation

* add some javadocs

* add some javadocs

* address review comments

* fix missing import

* address comments

* address review comments

* Update services/state/token/account.proto

Co-authored-by: Richard Bair <[email protected]>

* Update services/state/token/account.proto

Co-authored-by: Richard Bair <[email protected]>

* address some review comments

* move back under services

* change package name

* some docs and revert java package name

* revert docs

* add javadocs

* add javadocs

* re-order the first 16

---------

Signed-off-by: Michael Tinker <[email protected]>
Co-authored-by: Michael Tinker <[email protected]>
Co-authored-by: Richard Bair <[email protected]>
* 265 - file proto

Signed-off-by: Lev Povolotsky <[email protected]>

* 265 - packaging

Signed-off-by: Lev Povolotsky <[email protected]>

* 265 - comments

Signed-off-by: Lev Povolotsky <[email protected]>

---------

Signed-off-by: Lev Povolotsky <[email protected]>
* change allowances

* change allowances

* change allowances

* change allowances

* change allowances

* change allowances
* 265 - file proto

Signed-off-by: Lev Povolotsky <[email protected]>

* 265 - packaging

Signed-off-by: Lev Povolotsky <[email protected]>

* 265 - comments

Signed-off-by: Lev Povolotsky <[email protected]>

* 265 - comments fix

Signed-off-by: Lev Povolotsky <[email protected]>

---------

Signed-off-by: Lev Povolotsky <[email protected]>
* add missing fields and change allowance names

* change to bits

* address review comments

* address review comments

* address review comments
* Add bytecode and storage slot representations

Signed-off-by: Michael Tinker <[email protected]>

* Add Nft type

Signed-off-by: Michael Tinker <[email protected]>

* Add EntityNumber

Signed-off-by: Michael Tinker <[email protected]>

* Use bytes instead of Int256Value

Signed-off-by: Michael Tinker <[email protected]>

* Fix typo

Signed-off-by: Michael Tinker <[email protected]>

* Remove superfluous Int256Value type

Signed-off-by: Michael Tinker <[email protected]>

* Rename Id -> UniqueTokenId

Signed-off-by: Michael Tinker <[email protected]>

* Don't store code_hash, can be cached on load

Signed-off-by: Michael Tinker <[email protected]>

---------

Signed-off-by: Michael Tinker <[email protected]>
* Added state for blocks and running hashes

Signed-off-by: jasperpotts <[email protected]>

---------

Signed-off-by: jasperpotts <[email protected]>
Co-authored-by: jasperpotts <[email protected]>
@Neeharika-Sompalli Neeharika-Sompalli requested review from rbair23, tinker-michaelj and mhess-swl and removed request for a team, iwsimon and MarcKriguerAtHedera June 8, 2023 23:32
rbair23
rbair23 previously approved these changes Jun 9, 2023
mhess-swl
mhess-swl previously approved these changes Jun 9, 2023
Base automatically changed from add-pbj-types-for-state to main March 7, 2024 17:19
@nathanklick nathanklick dismissed stale reviews from mhess-swl and rbair23 March 7, 2024 17:19

The base branch was changed.

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.

6 participants