View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007730 | Taler | libeufin-bank-ui (SPA) | public | 2023-02-26 17:32 | 2023-09-23 15:09 |
| Reporter | Florian Dold | Assigned To | sebasjm | ||
| Priority | high | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | no change required | ||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0007730: demobank-ui should clear client state when server config has changed | ||||
| Description | We can easily detect this by storing a hash of the config. | ||||
| Tags | No tags attached. | ||||
|
|
waiting for 0007527 |
|
|
I don't think this is related to 0007527. This bug is about the configuration for the *ui* exclusively (which backend to use), and that is orthogonal to the LibEuFin configuration. |
|
|
> when server config has changed ACK, this confuses me. Then it should be the UI settings like backendBaseURL or allowRegistrations. Right? and not about the /config values of the backend. Like fee rates. |
|
|
Yes, exactly. It's *only* about globalThis.talerDemobankSettings, which are typically set in demobank-ui-settings.js. The problem is, right now, when these settings change, it often has no effect until localStorage is cleared. |
|
|
I don't think this is needed now. All the settings is read from the config file and the config file and it's not cached in the local storage as whole. The only attribute that is being cached is `bankUiSettings.backendBaseURL` with the local storage key `bank-base-url` which is not exposed in the UI. The only way to override the value is manually using the development tool from the browser. This was not the case with older version of the SPA, which used local storage for extensively. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2023-02-26 17:32 | Florian Dold | New Issue | |
| 2023-02-26 17:32 | Florian Dold | Status | new => assigned |
| 2023-02-26 17:32 | Florian Dold | Assigned To | => sebasjm |
| 2023-02-28 12:25 | sebasjm | Relationship added | related to 0007527 |
| 2023-02-28 12:26 | sebasjm | Note Added: 0019908 | |
| 2023-03-05 15:59 | Florian Dold | Priority | normal => high |
| 2023-03-05 16:00 | Florian Dold | Note Added: 0019921 | |
| 2023-03-05 19:51 | sebasjm | Relationship deleted | related to 0007527 |
| 2023-03-05 19:52 | sebasjm | Note Added: 0019922 | |
| 2023-03-05 20:32 | Florian Dold | Note Added: 0019923 | |
| 2023-03-27 19:09 | sebasjm | Status | assigned => resolved |
| 2023-03-27 19:09 | sebasjm | Resolution | open => no change required |
| 2023-03-27 19:09 | sebasjm | Note Added: 0019978 | |
| 2023-09-23 15:07 | Christian Grothoff | Fixed in Version | => 0.9.3 |
| 2023-09-23 15:09 | Christian Grothoff | Status | resolved => closed |
| 2023-12-03 01:21 | Christian Grothoff | Category | demobank-ui => libeufin-bank-ui |
| 2024-01-12 14:10 | Christian Grothoff | Category | libeufin-bank-ui => libeufin-bank-ui (SPA) |
