Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Get rid of LocalStorage #233

Open
1 of 2 tasks
ozgunozerk opened this issue Jun 1, 2022 · 1 comment
Open
1 of 2 tasks

Get rid of LocalStorage #233

ozgunozerk opened this issue Jun 1, 2022 · 1 comment
Assignees
@ozgunozerk
Labels
improvement it is already working, but can be better
Milestone
0.7.0

Comments

@ozgunozerk
Copy link
Contributor

ozgunozerk commented Jun 1, 2022 •
edited
Loading

This has been the root of many problems, yet we have another bug related to it.
If the user somehow changes their reward address (only way is to tinker with config file now), and starts from scratch by deleting every folder, the farmedBlocks panel will still show previously farmed blocks under the previous reward address.

  • For a quick fix, we should check these farmed blocks against the reward address, and only display the ones that are matching.
  • For a longer term fix, we should get rid of the LocalStorage completely imo.
@ozgunozerk ozgunozerk self-assigned this Jun 1, 2022
@ozgunozerk ozgunozerk added the bug Something isn't working label Jun 1, 2022
@ozgunozerk ozgunozerk moved this from Todo to In Progress in Product Engineering Jun 1, 2022
@ozgunozerk ozgunozerk added improvement it is already working, but can be better and removed bug Something isn't working labels Jun 14, 2022
@ozgunozerk ozgunozerk moved this from In Progress to Blocked in Product Engineering Jul 21, 2022
@ozgunozerk
Copy link
Contributor Author

The new dashboard design may not display the farmedBlocks, so this issue may be irrelevant in the future. Track the new dashboard from this issue: #238

@achiurizo achiurizo added this to the 0.7.0 milestone Nov 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ozgunozerk ozgunozerk

Labels
improvement it is already working, but can be better
Projects
Product Engineering
Status: Blocked
Milestone
0.7.0
Development

No branches or pull requests
2 participants
@achiurizo @ozgunozerk