Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync in worker #4

Closed
wants to merge 24 commits into from
Closed

Sync in worker #4

wants to merge 24 commits into from

Conversation

psrpinto
Copy link
Member

@psrpinto psrpinto commented Feb 14, 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

Required refactors

  • 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 14, 2023 15:17
@psrpinto psrpinto changed the title Session pool Client pool Feb 14, 2023
@psrpinto psrpinto changed the title Client pool Sync in worker Feb 20, 2023
@psrpinto
Copy link
Member Author

closing in favour of #6

@psrpinto psrpinto closed this Feb 20, 2023
@psrpinto psrpinto deleted the session-pool branch March 27, 2023 15:41
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