View Issue Details

IDProjectCategoryView StatusLast Update
0007730Talerlibeufin-bank-ui (SPA)public2023-09-23 15:09
ReporterFlorian Dold Assigned Tosebasjm  
PriorityhighSeverityminorReproducibilityhave not tried
Status closedResolutionno change required 
Target Version0.9.3Fixed in Version0.9.3 
Summary0007730: demobank-ui should clear client state when server config has changed
DescriptionWe can easily detect this by storing a hash of the config.
TagsNo tags attached.



2023-02-28 12:26

developer   ~0019908

waiting for 0007527

Florian Dold

2023-03-05 16:00

manager   ~0019921

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.


2023-03-05 19:52

developer   ~0019922

> 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.

Florian Dold

2023-03-05 20:32

manager   ~0019923

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.


2023-03-27 19:09

developer   ~0019978

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.

Issue History

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)