-
Notifications
You must be signed in to change notification settings - Fork 6
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
E2E test of Pente<->Noto integrated tokens with Bond sample #340
Merged
Conversation
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
Signed-off-by: Jim Zhang <jim.zhang@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
… into domain-conf
…nto block-timestamps
… into domain-conf
…nto block-timestamps
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
…confirmed TX states Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Jim Zhang <jim.zhang@kaleido.io>
Minor UI fix
Signed-off-by: John Hosie <john.hosie@kaleido.io>
sign on assembling node
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
…body Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
Apply new UI styles
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
… issue Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
Update dialog boxes
Validate state hashes for the zeto domain
Signed-off-by: dwertent <david.wertenteil@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Serve UI from ui path
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
Adding UI api to Paladin
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
hosie
reviewed
Nov 1, 2024
hosie
approved these changes
Nov 1, 2024
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.
LGTM
Signed-off-by: John Hosie <john.hosie@kaleido.io>
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Provides Domain-specific receipt information for completed transactions:
ptx_getTransactionReceipt
- no change to content, but now available on all nodesptx_getStateReceipt
- provides information about the theinput
/read
/output
/info
states for a TXptx_getDomainReceipt
- gives domain specific information about private transactionstransaction
with the Ethereum transaction andreceipt
with an Ethereum receiptptx_getTransactionReceiptFull
- includesstates
anddomainReceipt
domainReceiptError
if unavailable, rather than erroringptx_call
- now fully wired up for Pente private EVM contracts as well as public smart contractsptx_upsertABI
and sending txs):ptx_decodeError
ptx_decodeCall
ptx_decodeEvent
Includes significant work from @awrichar in demonstrating the use of a Pente private EVM smart contract for sophisticated notarization of a Noto based token modeling a Bond.
Detailed work list built as I went through
read
states through Proto, Domain and State ManagerUNION ALL
combined withRIGHT JOIN
construct to join things on the DB sidepsql
to debug now 😅DELETE FROM
instead ofDROP DB
between tests - ⏩ test executionrawTransaction
is what is written into the newinfo
stateevmVersion
,baseBlock
andbytecodeLen
which are the execution contextptx_call
for private transactions, and works through ERC-20 transfer E2E #344to_domain
proto interface to construct a receiptinfo
type of state emitted from a transactionEndorse
andReceipt
flows with the Domaindomain
to the receiptsptx_getTransactionReceiptFull
or similar interface to call thiscontractAddress
for deploys - unblocks E2E workfrom
informationfrom
into the context of deploys #353