-
Notifications
You must be signed in to change notification settings - Fork 5
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
[feat] Update get wallet Atom Hook in connectwallet.ts #21
Comments
Gm, can I work on this part of the project. I am proficient in TypeScript |
I'm Usman Adamu, a developer. |
May I try my hand at this? I'm a Software dev with over four years experience ETA: 5 hours |
I am a Full Stack Developer specializing in Next.js, TypeScript, Node.js, Cairo and Rust . With over 50 contributions across projects in the OnlyDust ecosystem, I’ve developed strong proficiency in delivering high-quality solutions and resolving complex issues within tight deadlines. My experience spans frontend, backend, smart contracts, and the optimization and maintenance of scalable codebases. |
Could I be assigned to this? |
Could I take on this issue? |
Could I take a shot at this? |
Hello, I am Joseph Chimebuka and I am an active software developer and a Diamond contributor on Onlydust I can handle this task can you please assign. I will create a PR in 48hrs |
Let me handle this issue! |
Can I try solving this issue? |
Could I try solving this? |
May I be assigned to this? |
May I try my hand at this? |
Let me handle this issue! |
Description
Refactor the walletStarknetkitLatestAtom in connectwallet.ts to use atomWithStorage for persistent storage instead of atomWithReset. This will ensure that the wallet information is stored and retrieved correctly across app sessions.
Requirements
• Replace the current atomWithReset implementation with atomWithStorage.
• Update the walletStarknetkitLatestAtom definition to store the value in persistent storage.
• New implementation should look like this:
• Run regression tests across the entire application to ensure that this change does not break any existing functionality.
Acceptance Criteria
• The walletStarknetkitLatestAtom is updated to use atomWithStorage correctly.
• Regression tests pass successfully across all parts of the app, with no issues related to this change.
• Persistent wallet data works as expected, preserving the state across sessions.
Important
Each commit should include the number of the issue to keep track of the issues, ex:
#[your-issue-number] - issue description
The text was updated successfully, but these errors were encountered: