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

Save Wallet Name and Url Image on creation #245

Open
danwag06 opened this issue Oct 4, 2024 · 6 comments
Open

Save Wallet Name and Url Image on creation #245

danwag06 opened this issue Oct 4, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@danwag06
Copy link
Collaborator

danwag06 commented Oct 4, 2024

The wallet currently supports giving different accounts a name (label) and a url icon. The request would be to optionally save this information when creating/restoring/importing the wallet.

@danwag06 danwag06 added enhancement New feature or request good first issue Good for newcomers labels Oct 4, 2024
@maxslimb
Copy link

maxslimb commented Oct 5, 2024

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

I have worked on typescript and JS before on various projects and latest I was working with Nethermind on the Voyager starknet block explorer

@danwag06
Copy link
Collaborator Author

danwag06 commented Oct 5, 2024

@maxslimb if you'd like to tackle this, see https://github.com/yours-org/yours-wallet/blob/main/CONTRIBUTING.md

When saving you'll want to do something similar to this to save the name and/or icon in chrome local storage:

const handleAccountEditSave = async () => {

There are a number of different ways a user can create/import/restore a wallet which can be found here:
https://github.com/yours-org/yours-wallet/tree/main/src/pages/onboarding

@maxslimb
Copy link

Hi @danwag06,
where can I find the account label, and the image URL is for the image associated with the account label?

Thanks

@danwag06
Copy link
Collaborator Author

@maxslimb Everything from an account state perspective is stored using chrome storage api.

To see this you can open the wallet in your browser window then:

  1. Right click and inspect the wallet window.
  2. In the console run chrome.storage.local.get(null, console.log)
  3. You can see the account label and the url image in accounts[account].settings.socialProfile

@maxslimb
Copy link

@danwag06 I just want to confirm, that we need to save the label which is "Account 1" if I create a new account and the URL image is the image shown near the label, for instance, the small circle.

@danwag06
Copy link
Collaborator Author

Yes correct @maxslimb

I should also tell you about a hackathon that's going on where you could get rewarded for this issue. It's happening via a fork of this repo but we plan to merge any changes that come from it. This issue is a part of the hackathon.

https://app.onlydust.com/hackathons/odhack-90

The project of interest will be spv-wallet-browser
https://app.onlydust.com/p/spv-wallet-browser

It's starts tomorrow. If I see your application come through I will approve it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants