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

Transfer lock to next oldest session when disconnecting from the builder #10789

Merged
merged 2 commits into from
Jun 5, 2023

Conversation

aptkingston
Copy link
Member

@aptkingston aptkingston commented Jun 5, 2023

Description

This PR adds a new feature to collaboration, so that whenever the "primary" user disconnects (the person who can edit screens and automations), this ability is automatically transferred to the user who has also been editing the same app the longest.

The prevents the need to either refresh your browser or exit the app and enter it again in order to be able to edit screens.

The user who gets control will get a notification letting them know they're able to edit screens and automations now, and their builder will automatically update itself.

This PR also slightly updates the websocket API to ensure that all payloads are wrapped in objects.

Addresses:

See it in action here:

Screen.Recording.2023-06-05.at.13.47.45.mov

@codecov-commenter
Copy link

Codecov Report

Merging #10789 (881fb63) into develop (a96d8ac) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff            @@
##           develop   #10789   +/-   ##
========================================
  Coverage    49.98%   49.98%           
========================================
  Files          153      153           
  Lines         5164     5164           
  Branches      1033     1033           
========================================
  Hits          2581     2581           
  Misses        2365     2365           
  Partials       218      218           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Collaborator

@mike12345567 mike12345567 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Actually quite a valuable change, does look like a very clean experience

Copy link
Member

@shogunpurple shogunpurple left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome, really slick. Thanks for making the update 🙏

@shogunpurple shogunpurple merged commit 0b7ed57 into develop Jun 5, 2023
@shogunpurple shogunpurple deleted the collaboration-lock-transfer branch June 5, 2023 16:47
@github-actions github-actions bot locked and limited conversation to collaborators Jun 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants