-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
feat(replay): Show CTA when org is not opted into gen ai features #95780
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
Merged
michellewzhang
merged 9 commits into
master
from
cursor/promote-ai-summary-tab-with-cta-a9b4
Jul 21, 2025
Merged
feat(replay): Show CTA when org is not opted into gen ai features #95780
michellewzhang
merged 9 commits into
master
from
cursor/promote-ai-summary-tab-with-cta-a9b4
Jul 21, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
af43cfc
to
e2a713d
Compare
Co-authored-by: billy <[email protected]>
Co-authored-by: billy <[email protected]>
e2a713d
to
58bf482
Compare
billyvg
commented
Jul 18, 2025
Comment on lines
-43
to
-67
const openForm = useFeedbackForm(); | ||
|
||
const feedbackButton = ({type}: {type: 'positive' | 'negative'}) => { | ||
return openForm ? ( | ||
<Button | ||
aria-label={t('Give feedback on the AI summary section')} | ||
icon={<IconThumb direction={type === 'positive' ? 'up' : 'down'} />} | ||
title={type === 'positive' ? t('I like this') : t(`I don't like this`)} | ||
size={'xs'} | ||
onClick={() => | ||
openForm({ | ||
messagePlaceholder: | ||
type === 'positive' | ||
? t('What did you like about the AI summary and chapters?') | ||
: t('How can we make the AI summary and chapters work better for you?'), | ||
tags: { | ||
['feedback.source']: 'replay_ai_summary', | ||
['feedback.owner']: 'replay', | ||
['feedback.type']: type, | ||
}, | ||
}) | ||
} | ||
/> | ||
) : null; | ||
}; |
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.
Refactored this into a new component <FeedbackButton>
The CTA doesn't exactly make sense because I think users need to manually contact us for access I think? But we can refine before next rollout. |
ryan953
approved these changes
Jul 18, 2025
michellewzhang
approved these changes
Jul 18, 2025
…tab-with-cta-a9b4
andrewshie-sentry
pushed a commit
that referenced
this pull request
Jul 21, 2025
…5780) There are a few flags at work here: - `replay-ai-summaries` is our rollout feature flag - `gen-ai-features` is config based feature flag to show AI features - `organization.hideAiFeatures` is a user/org opt-in to gen AI features (similar to above flag) - and finally there is the org opt-in/acknowledgement to Seer features The AI Summary tab is now visible if only the rollout flag `replay-ai-summaries` and the gen AI flags are enabled . If organization can see gen AI features but has *not* opted into Seer, the AI Summary panel will display a CTA guiding users to their organization settings to enable Seer (or get info on how to enable it). <img width="627" height="200" alt="image" src="https://github.com/user-attachments/assets/a34cab3a-a15e-4718-82a8-f6ddde2b9deb" /> Closes REPLAY-526 --------- Co-authored-by: Cursor Agent <[email protected]> Co-authored-by: Michelle Zhang <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There are a few flags at work here:
replay-ai-summaries
is our rollout feature flaggen-ai-features
is config based feature flag to show AI featuresorganization.hideAiFeatures
is a user/org opt-in to gen AI features (similar to above flag)The AI Summary tab is now visible if only the rollout flag
replay-ai-summaries
and the gen AI flags are enabled .If organization can see gen AI features but has not opted into Seer, the AI Summary panel will display a CTA guiding users to their organization settings to enable Seer (or get info on how to enable it).
Closes REPLAY-526