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 token service #287

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

Conversation

Neeharika-Sompalli
Copy link
Member

No description provided.

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

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

* Add running hash, delete EntityNumber

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

* 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 <rbair23@users.noreply.github.com>

* Update services/state/token/account.proto

Co-authored-by: Richard Bair <rbair23@users.noreply.github.com>

* 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 <michael.tinker@swirldslabs.com>
Co-authored-by: Michael Tinker <michael.tinker@swirldslabs.com>
Co-authored-by: Richard Bair <rbair23@users.noreply.github.com>
* 265 - file proto

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>

* 265 - packaging

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>

* 265 - comments

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>

---------

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>
* change allowances

* change allowances

* change allowances

* change allowances

* change allowances

* change allowances
* 265 - file proto

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>

* 265 - packaging

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>

* 265 - comments

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>

* 265 - comments fix

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>

---------

Signed-off-by: Lev Povolotsky <lev@swirldslabs.com>
* 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 <michael.tinker@swirldslabs.com>

* Add Nft type

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

* Add EntityNumber

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

* Use bytes instead of Int256Value

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

* Fix typo

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

* Remove superfluous Int256Value type

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

* Rename Id -> UniqueTokenId

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

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

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>

---------

Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
* Added state for blocks and running hashes

Signed-off-by: jasperpotts <jasperpotts@users.noreply.github.com>

---------

Signed-off-by: jasperpotts <jasperpotts@users.noreply.github.com>
Co-authored-by: jasperpotts <jasperpotts@users.noreply.github.com>
@Neeharika-Sompalli Neeharika-Sompalli requested review from a team as code owners June 23, 2023 15:16
@Neeharika-Sompalli Neeharika-Sompalli requested review from netopyr, MarcKriguerAtHedera, iwsimon and a team and removed request for a team and MarcKriguerAtHedera June 23, 2023 15:16
iwsimon
iwsimon previously approved these changes Jun 23, 2023
Copy link
Contributor

@iwsimon iwsimon left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from add-pbj-types-for-state to main March 7, 2024 17:19
@nathanklick nathanklick dismissed iwsimon’s stale review 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.

None yet

6 participants