-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Free trial popover #5926
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
Free trial popover #5926
Conversation
Your cubic subscription is currently inactive. Please reactivate your subscription to receive AI reviews and use cubic. |
✅ Deploy Preview for continuedev canceled.
|
On the flip side I can also see having it in
|
|
||
const { profiles, session, login, selectedProfile, refreshProfiles } = | ||
useAuth(); | ||
|
||
// If using free trial, show the FreeTrialButton instead | ||
const isUsingFreeTrial = usesFreeTrialApiKey(config); | ||
if (isUsingFreeTrial) { |
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.
I would move this logic into a ternary in the parent component, gui/src/components/mainInput/Lump/LumpToolbar/BlockSettingsTopToolbar.tsx
|
||
useEffect(() => { | ||
ideMessenger | ||
.request("controlPlane/getFreeTrialStatus", undefined) |
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 should try to use async
/await
instead of callback chaining. Assuming this was AI generated code, could be a good rule 👀
Description
Make it more clear to users that they are using the free trial, and how to proceed past it
Checklist
Tests
tbd