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-1126] Remove PrivateKey type from wallet #18458

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

Quexington
Copy link
Contributor

@Quexington Quexington commented Aug 13, 2024

This PR attempts to remove the PrivateKey type from the wallet insofar as is possible. The object is to be able to use any type of private key with the wallet's .private_key member variable.

@Quexington Quexington force-pushed the quex.remove_bls_only_from_wallet branch from 9bfb7f7 to 586cab7 Compare August 13, 2024 20:15
@Quexington Quexington added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Aug 13, 2024
@Quexington Quexington changed the title Remove PrivateKey type from wallet [CHIA-1126] Remove PrivateKey type from wallet Aug 14, 2024
Base automatically changed from quex.add_secret_info_protocol to long_lived/vault August 14, 2024 17:02
@Quexington Quexington marked this pull request as ready for review August 14, 2024 17:03
@Quexington Quexington requested a review from a team as a code owner August 14, 2024 17:03
@Quexington Quexington closed this Aug 14, 2024
@Quexington Quexington reopened this Aug 14, 2024
@Quexington Quexington closed this Aug 16, 2024
@Quexington Quexington reopened this Aug 16, 2024
@Quexington Quexington closed this Aug 19, 2024
@Quexington Quexington reopened this Aug 19, 2024
@Quexington Quexington merged commit 58d25b1 into long_lived/vault Aug 19, 2024
1475 of 1478 checks passed
@Quexington Quexington deleted the quex.remove_bls_only_from_wallet branch August 19, 2024 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes vault
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants