-
Notifications
You must be signed in to change notification settings - Fork 44
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
On-chain accounts protobuf & domain objects #287
Merged
Merged
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
b6c529f
feat: add `account_details` table to the DB
polydez ed04d5a
refactor: rename `block_number` column in nullifiers table to `block_…
polydez 4cc4ba8
refactor: use `BETWEEN` in interval comparison checks
polydez 8f0935a
feat: implement account details protobuf messages, domain objects and…
polydez 9f5c832
feat: (WIP) implement account details support
polydez 58dc518
feat: (WIP) implement account details support
polydez ef7a1f7
feat: (WIP) implement account details support
polydez bb31753
feat: (WIP) implement account details support
polydez ac81502
fix: db creation
polydez ab9c457
docs: remove TODO
polydez 31cfd19
refactor: apply formatting
polydez 6560444
feat: implement endpoint for getting public account details
polydez bd7ac74
Merge branch 'next' into polydez-onchain-accounts
polydez be2be66
tests: add test for storage
polydez fbcfd6b
feat: add rpc endpoint for getting account details
polydez 28d44fb
refactor: keep only domain object changes
polydez d344c57
Merge branch 'next' into polydez-onchain-accounts
polydez 618e24e
fix: compilation errors
polydez bd47398
fix: use note tag conversion from `u64`
polydez 42b0c22
refactor: remove account details protobuf messages
polydez bb6e18e
fix: remove unused error invariants
polydez 535797e
refactor: introduce `UpdatedAccount` struct
polydez 3266818
fix: rollback details conversion
polydez fcdc331
fix: compilation error
polydez 6eafe6e
Merge branch 'next' into polydez-onchain-accounts
polydez 20f0756
format: reformat using rustfmt
polydez File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should add the missing conversions to
miden-base
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hackaugusto nice idea, done: 0xPolygonMiden/miden-base#559
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eventually (in a different PR), tag here would be
u32
as well, right