-
Notifications
You must be signed in to change notification settings - Fork 215
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
Feature/real time usage update #807
Conversation
apps/dashboard/web/polaris_web/web/src/apps/dashboard/pages/settings/billing/Billing.jsx
Outdated
Show resolved
Hide resolved
@@ -566,6 +566,11 @@ public void startTestNew(ApiInfo.ApiInfoKey apiInfoKey, ObjectId testRunId, | |||
if (!applyRunOnceCheck(apiInfoKey, testConfig, subCategoryEndpointMap, apiInfoKeyToHostMap, testSubCategory)) { | |||
continue; | |||
} | |||
|
|||
if(syncLimit.updateUsageLeftAndCheckSkip()){ | |||
continue; |
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.
[Minor] Add a log here, print usage details
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.
@notshivansh lets add this?
boolean isGranted; | ||
public static final String IS_GRANTED = "isGranted"; | ||
public int overageFirstDetected = -1; | ||
int overageFirstDetected = -1; | ||
public static final String OVERAGE_FIRST_DETECTED = "overageFirstDetected"; |
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.
We can keep access modifiers of instance vars to be private if they have apt getters and setters
No description provided.