-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Fix default workspace class is not selected #12436
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
Conversation
started the job as gitpod-build-furisto-default-workspace-class-12435.2 because the annotations in the pull request description changed |
0004d16
to
51fc7bd
Compare
@@ -265,6 +266,9 @@ const gitpodServiceMock = createServiceMock({ | |||
}, | |||
}; | |||
}, | |||
getBillingModeForUser: async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems unrelated to the PR, why is this change needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The settings page calls this method and if it is not there the page cannot be displayed when using the service mock.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Will require a rebase
51fc7bd
to
a93f029
Compare
Description
Show the workspace class that will be used if no workspace class has been selected yet by the user.
Related Issue(s)
Fixes #12435
How to test
Release Notes
Werft options: