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

Fix(plugins): support multiple users #930

Merged
merged 3 commits into from
Dec 8, 2021
Merged

Fix(plugins): support multiple users #930

merged 3 commits into from
Dec 8, 2021

Conversation

imsnif
Copy link
Member

@imsnif imsnif commented Dec 8, 2021

This is another layer of preparation for supporting multiple users.

In this episode, we:

  1. Change plugins so that one is created for each user
  2. Change the infrastructure around them so that plugins are individually updated by user actions (rather than all relevant plugins be updated for each relevant user action)
  3. Change the various input mode tracking around the app so that it's per client. This has the upshot that even not in mirrored sessions, the plugins are unique per client (which provides for a much nicer user experience).

@imsnif imsnif merged commit 92bddf1 into main Dec 8, 2021
@har7an har7an deleted the plugin-multiple-users branch October 23, 2022 15:10
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