Skip to content

Conversation

@psrpinto
Copy link
Member

@psrpinto psrpinto commented Feb 20, 2023

WIP

This is an exploration, this PR is not intended to be merged. This might result in a PR to upstream hydrogen, or we could end up dropping it entirely.

Related to Automattic/chatrix#159

TODO

  • Make olm functional in sync worker
  • Make sync in a worker functional, without caring about "forwarding" state to the UI thread
  • "Forward" state from sync worker to UI thread

Refactors that could be necessary and/or nice-to-have

  • Rename WorkerPool and related files/classes to OlmWorkerPool
  • Base Worker abstraction
  • Improve worker-related directory structure
  • Have vite handle worker paths

@psrpinto psrpinto marked this pull request as draft February 20, 2023 13:19
@psrpinto psrpinto mentioned this pull request Feb 20, 2023
@psrpinto
Copy link
Member Author

psrpinto commented Mar 7, 2023

Closing since proposal has been submitted upstream. Will leave the branch around for the time being, for reference.

@psrpinto psrpinto closed this Mar 7, 2023
@psrpinto
Copy link
Member Author

psrpinto commented Mar 7, 2023

Renamed branch to sync-in-worker-poc

@psrpinto psrpinto deleted the sync-in-worker branch March 7, 2023 14:02
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.

2 participants