View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0007446 | Taler | wallet-core | public | 2022-11-07 16:30 | 2023-09-23 15:09 |
Reporter | sebasjm | Assigned To | sebasjm | ||
Priority | normal | Severity | feature | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Product Version | 0.9.1 | ||||
Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
Summary | 0007446: dev mode is not persisted after restart | ||||
Description | Service worker is a short lived process, so any setting on the wallet core should persist restart. | ||||
Tags | No tags attached. | ||||
|
should we add this into the wallet-core db? |
|
Does it even make sense to persist dev-mode? I actually think turning it off on restart might be a good thing. Devs can re-enable, but having it on by accident might be bad... |
|
On iOS, Dev-Mode is persisted in the application settings, which is just two words of code preceding a variable definition: @AppStorage("developerMode") var developerMode: Bool = false Next time the app launches it will not init with false, but use the value for key "developerMode". If I delete the first two words, the app will fall back to false at launch - after reboot, or if the user manually kicked it off the background app list, or the user launched so many other apps while we are in the background that iOS runs out of memory and kills us - which could take days if the user doesn't launch many other apps. In this case they would maybe see Dev-Mode active for multiple days, and on other days falling back to inactive even within minutes (if they launch many other apps before coming back to us). Such unpredictability is not the best user experience - but OK, it's just developer mode... |
|
> I actually think turning it off on restart might be a good thing. Restart the browser yes, but service worker may restart (wiping out wallet memory) after short time. I will implement saving the state in localStorage (the web version of marc's description) and sync wallet on startup. |
|
fixed with the implementation of feature flags |
Date Modified | Username | Field | Change |
---|---|---|---|
2022-11-07 16:30 | sebasjm | New Issue | |
2022-11-07 16:30 | sebasjm | Status | new => assigned |
2022-11-07 16:30 | sebasjm | Assigned To | => Florian Dold |
2022-11-16 18:55 | sebasjm | Note Added: 0019428 | |
2022-11-18 22:40 | Christian Grothoff | Note Added: 0019452 | |
2022-11-23 00:07 | MarcS | Note Added: 0019473 | |
2022-11-23 12:57 | sebasjm | Note Added: 0019477 | |
2022-11-23 12:57 | sebasjm | Assigned To | Florian Dold => sebasjm |
2023-01-13 17:08 | Christian Grothoff | Severity | minor => feature |
2023-01-13 17:08 | Christian Grothoff | Target Version | git (master) => 0.9.4 |
2023-04-13 20:36 | Florian Dold | Category | wallet (TS core) => wallet-core |
2023-04-26 16:28 | sebasjm | Status | assigned => resolved |
2023-04-26 16:28 | sebasjm | Resolution | open => fixed |
2023-04-26 16:28 | sebasjm | Note Added: 0020163 | |
2023-04-29 13:49 | Christian Grothoff | Fixed in Version | => 0.9.3 |
2023-04-29 13:49 | Christian Grothoff | Target Version | 0.9.4 => 0.9.3 |
2023-09-23 15:09 | Christian Grothoff | Status | resolved => closed |