Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Change link URL and label, and display only for Pro #61873

Merged
merged 4 commits into from
Apr 18, 2024

Conversation

vdavid
Copy link
Contributor

@vdavid vdavid commented Apr 15, 2024

Closes https://github.com/sourcegraph/self-serve-cody/issues/510

This is what it used to look like:
Before

This is what it looks like now for Pro users:

After - Pro users

And for Free users:

After - Free users

Test plan

Checked out https://sourcegraph.test:3443/cody/manage with both a Pro and a Free user, to see the diff. See the screenshots above.

@cla-bot cla-bot bot added the cla-signed label Apr 15, 2024
@sourcegraph-bot
Copy link
Contributor

sourcegraph-bot commented Apr 15, 2024

📖 Storybook live preview

@vdavid vdavid requested review from a team and thenamankumar April 15, 2024 10:06
Copy link
Member

@thenamankumar thenamankumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vdavid vdavid enabled auto-merge (squash) April 15, 2024 11:01
@vdavid vdavid merged commit a73ef24 into main Apr 18, 2024
@vdavid vdavid deleted the dv/add-cody-pro-support-link branch April 18, 2024 22:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants