Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

[Multi-accounts]: Handle multiple keystore based accounts #2334

Merged
merged 5 commits into from
Jul 29, 2022
Merged

[Multi-accounts]: Handle multiple keystore based accounts #2334

merged 5 commits into from
Jul 29, 2022

Conversation

veado
Copy link
Collaborator

@veado veado commented Jul 21, 2022 •
edited
Loading

  • Refactor/simplify KeystoreState
  • Handle multiple keystore/accounts in a single file accounts.json
  • Sync state (selected, name) of multiple keystores
  • Support migration of legacy keystore file, which is used by prev. versions of ASGDX
  • Moved optionFromNullableString from src/renderer/helpers/fp/from.ts to src/shared/utils/fp.ts

Part of #1601

@veado veado self-assigned this Jul 21, 2022
@veado veado marked this pull request as draft July 21, 2022 14:59
@veado veado changed the title WIP: Handle multiple keystores [Multi-accounts]: Handle multiple keystore files Jul 21, 2022
@veado
Continue on multiple keystores + accounts
20e95b5
@veado
Refactor keystore/account handling to simplify
be4696c 
things: One file only to store keystores + accounts. Most logic stays in
renderer thread, while main thread just handles persistent data on disk
creating/updating account.json file. Still WIP.
@veado veado changed the title [Multi-accounts]: Handle multiple keystore files [Multi-accounts]: Handle multiple keystore based accounts Jul 27, 2022
@veado veado marked this pull request as ready for review July 29, 2022 15:57
@veado veado merged commit 9d1f85d into develop Jul 29, 2022
@veado veado deleted the issue-1601/multi-keystores branch July 29, 2022 16:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@veado veado

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@veado