-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Stop sign on "out of credits" #11576
Conversation
/werft run 👍 started the job as gitpod-build-at-out-of-credits.2 |
/werft run 👍 started the job as gitpod-build-at-out-of-credits.3 |
ee99a1c
to
8ec9f06
Compare
4a8a0a8
to
99dfe13
Compare
99dfe13
to
332f0d4
Compare
// TODO(at) replace the naive implementation based on usage service | ||
// with a proper call check against the upcoming invoice. | ||
// For now this should just enable the work on fronend. | ||
if (await this.isUsageBasedFeatureFlagEnabled(user)) { |
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.
💯
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 🕺
@AlexTugarev Just noticed it requires a rebase 🙄 |
332f0d4
to
ded94dc
Compare
Description
Partially resolves #11404
Very limited UI 🛹 of a modal shown when you've reached the configured spending limit.
Related Issue(s)
Related to #11404
How to test
Release Notes
Documentation
Werft options: