Skip to content

[DRAFT] revert redux state registry and session#3407

Draft
panaC wants to merge 2 commits intodevelopfrom
revert-registry-session
Draft

[DRAFT] revert redux state registry and session#3407
panaC wants to merge 2 commits intodevelopfrom
revert-registry-session

Conversation

@panaC
Copy link
Member

@panaC panaC commented Feb 24, 2026

This commit introduce the revert of the redux win.session mechanism (currently broken) to save the current session (one library, multiple reader) at closing.

This remove also the reader registry persistence from the redux store, to rely on fileSystem publication directory index (appData/config-data/reader//...). Publication data preference will not be written in redux state in main process anymore, but instead written directly the disk. This allow to save massive data IO redux state persistence to disk by successive patches and final entire state dump.

next steps:

  • migrate registry to appData config-data/reader/ directory
  • handle library bound in appData/config-data by getter/setter functions in main/di
  • read/write publication state bound/config/locator.json to appData config-data/reader/

This commit introduce the revert of the redux win.session mechanism (currently broken)
to save the current session (one library, multiple reader) at closing.

This remove also the reader registry persistence from the redux store,
to rely on fileSystem publication directory index (appData/config-data/reader/<uuid>/...).
Publication data preference will not be written in redux state in main
process anymore, but instead written directly the disk.
This allow to save massive data IO redux state persistence to disk by successive
patches and final entire state dump.

next steps:

- migrate registry to appData config-data/reader/<uuid> directory
- handle library bound in appData/config-data by getter/setter functions in main/di
- read/write publication state bound/config/locator.json to appData config-data/reader/<uuid>
@panaC panaC self-assigned this Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant