Feat: Add SP Wallets and Update/Init Wallets #520
Merged
Conversation
This file contains hidden or 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
Adds command and API endpoint to initialize a newly created wallet with the ActorID on chain. Once a wallet is created with `singularity wallet create` and it's been sent Fil or DataCap, run `singularity wallet init <address>` to run a lookup and update it's ActorID field.
Adds the ability to update name, location, and contact fields for wallets. Fields like address, actor ID, balances, and type may not be updated.
Allows creating wallets of SPWallet type from create handler. This will validate the provided Address and ActorID to prevent errors. Also allows inserting optional details like name, contact info, and location on creation rather than requiring a separate update.
Sankara-Jefferson
deleted the branch
data-preservation-programs:zach-wallet-create
| require.NotNil(t, resp.Payload) | ||
| }) | ||
| t.Run("ListWallets", func(t *testing.T) { | ||
| resp, err := client.Wallet.ListWallets(&wallet2.ListWalletsParams{ |
Contributor
There was a problem hiding this comment.
Missing test for UpdateWallet.
Sankara-Jefferson
merged commit 8219668
into
data-preservation-programs:zach-wallet-create
31 of 36 checks passed
Sankara-Jefferson
pushed a commit
that referenced
this pull request
Jun 27, 2025
parkan
pushed a commit
that referenced
this pull request
Oct 22, 2025
- Adds ability to create wallet with optional details: `singularity wallet create --name "my wallet"` - Adds ability to create SP wallet: `singularity wallet create --address f12345 --actor-id f0123` - Handles all edge cases if parameters are not correct - Validates addresses and actor ID - Adds ability to modify details of any wallet: `singularity wallet update --name "new name" --location "somewhere else" --contact "their@email.com"` - Adds ability to update wallet after initialization: `singularity wallet init f12345` Relies on #517 The lint error is addressed by PR #523.
parkan
pushed a commit
that referenced
this pull request
Oct 22, 2025
parkan
pushed a commit
that referenced
this pull request
Nov 6, 2025
- Adds ability to create wallet with optional details: `singularity wallet create --name "my wallet"` - Adds ability to create SP wallet: `singularity wallet create --address f12345 --actor-id f0123` - Handles all edge cases if parameters are not correct - Validates addresses and actor ID - Adds ability to modify details of any wallet: `singularity wallet update --name "new name" --location "somewhere else" --contact "their@email.com"` - Adds ability to update wallet after initialization: `singularity wallet init f12345` Relies on #517 The lint error is addressed by PR #523.
parkan
pushed a commit
that referenced
this pull request
Nov 6, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
singularity wallet create --name "my wallet"singularity wallet create --address f12345 --actor-id f0123singularity wallet update --name "new name" --location "somewhere else" --contact "their@email.com"singularity wallet init f12345Relies on #517