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

[CHIA-307] (long-lived) Vault #16610

Draft
wants to merge 336 commits into
base: main
Choose a base branch
from
Draft

[CHIA-307] (long-lived) Vault #16610

wants to merge 336 commits into from

Conversation

Quexington
Copy link
Contributor

@Quexington Quexington commented Oct 12, 2023

Todo:

@Quexington Quexington added Added Required label for PR that categorizes merge commit message as "Added" for changelog vault labels Oct 12, 2023
@Chia-Network Chia-Network deleted a comment from coveralls-official bot Oct 19, 2023
@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Oct 30, 2023
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the merge_conflict Branch has conflicts that prevent merge to main label May 7, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot May 7, 2024
Copy link
Contributor

github-actions bot commented May 7, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label May 7, 2024
This adds a new command decorator (unused for now but more PRs to come)
that automatically adds the functionality to export transactions rather
than automatically push them.
@Quexington Quexington changed the title (long-lived) Vault [CHIA-307] (long-lived) Vault May 14, 2024
This adds the new @tx_out_cmd decorator to the `chia wallet send`
command. This allows the command to output a transaction file instead of
pushing the transaction automatically.
…long_lived_vault_from_main_bd3a638db10ee92a15f138ceaef06648c3c7b480
This brings another set of commands to the `@tx_out_cmd` decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.
This brings another set of commands to the @tx_out_cmd decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.
This brings another set of commands to the @tx_out_cmd decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.
This brings another set of commands to the @tx_out_cmd decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.
This brings another set of commands to the @tx_out_cmd decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.
This brings another set of commands to the @tx_out_cmd decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.
This brings another set of commands to the @tx_out_cmd decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.
…ling) (#18065)

This brings another set of commands to the @tx_out_cmd decorator which
gives it the capability to optionally push a transaction and export the
transactions to a local file.

This PR being the last in a chain of similar PRs, I have included some
comments explaining that DL and pooling endpoints will not be ported to
the decorator at this time. Didn't think that necessitated its own PR
and CI run :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Added Required label for PR that categorizes merge commit message as "Added" for changelog merge_conflict Branch has conflicts that prevent merge to main vault
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants