feat(seer): Add setup CTA in sub settings#104718
Conversation
github-actions
bot
added
the
Scope: Frontend
There was a problem hiding this comment.
@ryan953 i think you also needed to use this endpoint, so i figured it was worth making this into a hook
There was a problem hiding this comment.
yes i do!
I think there's a few PRs that add the hook, some flavor of it. I'll do a pass after and refactor them all to so we have a single copy of it. IMO that'll be easier than creating merge-conflicts first.
There was a problem hiding this comment.
thank you!!
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
static/gsApp/views/subscriptionPage/usageOverview/components/panel.tsx
Outdated
Show resolved
Hide resolved
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
| const {data: setupCheck, isLoading: setupCheckLoading} = | ||
| useSeerOnboardingCheck(shouldCheckSetup); |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
did this intentionally; if we can't get a successful response from the endpoint, i chose to fallback to assuming setup is still required
| <Container> | ||
| <Image | ||
| src={seerConfigMainImg} | ||
| alt={t('Find and fix issues anywhere with Seer AI debugger')} |
There was a problem hiding this comment.
do we need the alt if it's the same as the text in line 135?
There was a problem hiding this comment.
the component requires it lol
i suppose it could just be an empty string since it's purely decorative
3 participants
Implements setup required state for Seer product panel in Subscription Settings:
Also adds image for trial/upgrade CTA:
Also fixed a couple issues with CTAs: