Skip to content

feat: add frontend support for enabling automatic workspace updates #10375

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

Merged
merged 11 commits into from
Oct 31, 2023

Conversation

sreya
Copy link
Collaborator

@sreya sreya commented Oct 21, 2023

This PR adds support for configuring an update policy for a workspace. Some behavior notes:

  • A tooltip is displayed in the workspace stats bar if your update policy is overriden by the template-level setting.
  • As long as the template policy does not mandate that the updates are enforced, the restart/start buttons will appear on the dashboard (and do not update to the latest version) regardless of whether you've toggled the workspace-level setting. It feels wrong to restrict the options of the user based on a setting that they personally decide to opt-in or not.

Workspace settings page:

Screenshot 2023-10-21 at 5 15 29 PM

Workspace Page (Regular User) -- Policy=Never TemplatePolicy=Always:

Screenshot 2023-10-21 at 5 32 13 PM

Workspace Page (Regular User) -- Policy=Always TemplatePolicy=Disabled:

Screenshot 2023-10-21 at 5 34 07 PM

fixes #7465

@sreya sreya requested a review from aslilac October 21, 2023 22:21
@sreya sreya force-pushed the jon/wsautoupdatefe branch from 096ade4 to ce25bba Compare October 21, 2023 22:39
@sreya
Copy link
Collaborator Author

sreya commented Oct 21, 2023

@aslilac the one issue I'm seeing is that after I submit the form in the workspace settings page the whole form stays disabled until I refresh the page 🤔

@github-actions github-actions bot added the stale This issue is like stale bread. label Oct 29, 2023
@aslilac aslilac removed the stale This issue is like stale bread. label Oct 30, 2023
@aslilac
Copy link
Member

aslilac commented Oct 30, 2023

I can't figure out how to reproduce the disabled form, but I'm happy to have a quick debugging session later

@sreya sreya requested a review from aslilac October 30, 2023 23:44
import { Workspace } from "api/typesGenerated";
import {
AutomaticUpdates,
AutomaticUpdateses,
Copy link
Member

Choose a reason for hiding this comment

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

this name sounds like we hired gollum 😂

Copy link
Member

@aslilac aslilac left a comment

Choose a reason for hiding this comment

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

oh, I just noticed that this broke one of the storybook tests. check chromatic, but after that's fixed it should be good to go.

@sreya sreya requested a review from aslilac October 31, 2023 19:28
@sreya sreya merged commit f4026ed into main Oct 31, 2023
@sreya sreya deleted the jon/wsautoupdatefe branch October 31, 2023 22:06
@github-actions github-actions bot locked and limited conversation to collaborators Oct 31, 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.

workspace "automatic update" front end settings
3 participants