Skip to content

Conversation

eamodio
Copy link
Member

@eamodio eamodio commented Sep 9, 2024

This still needs finalized messaging and promo testing.

Copy link
Contributor

@axosoft-ramint axosoft-ramint left a comment

Choose a reason for hiding this comment

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

Looking good, just a couple of questions/suggestions.

I'm also wondering how we want the indicator to work once this is live. Would it still show up, grab and summarize PRs in the hover, etc.?

Also, would it be weird that the hover says "connect an integration to get started" when really the user needs to upgrade to Pro first?

@axosoft-ramint axosoft-ramint self-assigned this Sep 10, 2024
@eamodio eamodio force-pushed the feature/launchpad-exit-preview branch from fec8923 to 9bcdbeb Compare September 11, 2024 21:20
@axosoft-ramint axosoft-ramint self-requested a review September 11, 2024 23:54
eamodio and others added 3 commits September 11, 2024 20:58
 - Adds a promotion for Launchpad
 - Allows access until promo ends
Removes (disables) legacy "focus" editor
@axosoft-ramint axosoft-ramint force-pushed the feature/launchpad-exit-preview branch from 9bcdbeb to fdd22f3 Compare September 12, 2024 05:11
@axosoft-ramint axosoft-ramint marked this pull request as ready for review September 12, 2024 05:12
Copy link
Contributor

@axosoft-ramint axosoft-ramint left a comment

Choose a reason for hiding this comment

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

Tested as many cases as I could think of - LGTM

@axosoft-ramint axosoft-ramint merged commit 107c129 into main Sep 12, 2024
2 checks passed
@axosoft-ramint axosoft-ramint deleted the feature/launchpad-exit-preview branch September 12, 2024 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants