Skip to content

refactor(web): optimize ws client routing #810

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
Merged
merged 2 commits into from
Sep 5, 2023
Merged

refactor(web): optimize ws client routing #810

merged 2 commits into from
Sep 5, 2023

Conversation

@m8vago
Copy link
Contributor

@m8vago m8vago commented Sep 4, 2023

  • Use the routes map on client side websocket to route messages.
  • Fix deployment environment editing

@m8vago m8vago requested a review from a team as a code owner September 4, 2023 16:54
@github-actions github-actions bot added source:web The scope of the issue or pull request is web. lang:typescript pr:refactor labels Sep 4, 2023
polaroi8d
polaroi8d requested changes Sep 4, 2023
View reviewed changes
@m8vago m8vago requested a review from polaroi8d September 5, 2023 11:37
@polaroi8d polaroi8d merged commit 81aee61 into develop Sep 5, 2023
@polaroi8d polaroi8d deleted the refactor/optimize-ws-client-routing branch September 5, 2023 11:40
chandhuDev pushed a commit to chandhuDev/dyrectorio that referenced this pull request Nov 7, 2023
@m8vago @chandhuDev
refactor(web): optimize ws client routing (dyrector-io#810)
006bebb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@robot9706 robot9706 robot9706 approved these changes

@polaroi8d polaroi8d polaroi8d approved these changes

Assignees

@m8vago m8vago

Labels

lang:typescript pr:refactor source:web The scope of the issue or pull request is web.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@m8vago @robot9706 @polaroi8d