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

Extract a penumbra-shielded-pool crate out of app,transaction. #2434

Merged
merged 4 commits into from
Apr 29, 2023

Conversation

hdevalence
Copy link
Member

Part of #2288. This is a proof-of-possibility for the architecture described in that issue. Now, the penumbra-shielded-pool crate is self-contained, with both the on-chain structures required to track the shielded pool, and the transaction actions needed to interact with it.

In the future, a fair amount of functionality currently in the penumbra-crypto crate could now be lifted up into this crate, for instance the definitions of the spend and output proof statements.

Part of #2288.  This is a proof-of-possibility for the architecture described
in that issue.  Now, the `penumbra-shielded-pool` crate is self-contained, with
both the on-chain structures required to track the shielded pool, and the
transaction actions needed to interact with it.

In the future, a fair amount of functionality currently in the
`penumbra-crypto` crate could now be lifted up into this crate, for instance
the definitions of the spend and output proof statements.
@hdevalence hdevalence temporarily deployed to smoke-test April 29, 2023 21:24 — with GitHub Actions Inactive
@hdevalence hdevalence temporarily deployed to smoke-test April 29, 2023 21:37 — with GitHub Actions Inactive
@hdevalence hdevalence force-pushed the split-app-shieldedpool branch from 33ddf46 to 236f352 Compare April 29, 2023 21:53
@hdevalence hdevalence temporarily deployed to smoke-test April 29, 2023 21:53 — with GitHub Actions Inactive
@hdevalence hdevalence merged commit 8904276 into main Apr 29, 2023
@hdevalence hdevalence deleted the split-app-shieldedpool branch April 29, 2023 22:14
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.

1 participant