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.

Sign up for GitHub

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

Jump to bottom

Utility to debounce room creation #332

Open
Half-Shot opened this issue Jun 9, 2021 · 0 comments · May be fixed by #338
Open

Utility to debounce room creation #332

Half-Shot opened this issue Jun 9, 2021 · 0 comments · May be fixed by #338
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.

Comments

@Half-Shot
Copy link
Contributor

I've found that we often create rooms asynchronously and build in debouncing logic all over the place to avoid the same room being created twice. I'd suggest we create a debouncing component that simply keys a room create promise to a unique key, to avoid creating multiple rooms.
@Half-Shot Half-Shot linked a pull request Jul 15, 2021 that will close this issue
Open
@Half-Shot Half-Shot added T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements. and removed enhancement labels May 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add debounceRequest function matrix-org/matrix-appservice-bridge
1 participant
@Half-Shot