Skip to content
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

feat: implement a What's New prompt #539

Merged

Conversation

Abdomash
Copy link
Contributor

@Abdomash Abdomash commented Feb 28, 2025

This PR closes #529.

Implements the What's New popup

This creates a prompt that will show when the user opens the calendar view for the first time after the extension is updated or installed. The figma reference here.

Screenshot

image

TODO:

  • Use a real image/video instead of the placeholder image
  • Make changelog button navigate to settings page (TBD due to github-fetching lag in the settings page)
  • BUG: the dialog shouldn't reopen after opening the changelog (TBD might require changing Dialog.tsx)
  • Correctly fetch extension version from manifest

This change is Reviewable

@DereC4
Copy link
Member

DereC4 commented Mar 1, 2025

Don't we already have this

@Samathingamajig Samathingamajig added this to the v2.2.0 milestone Mar 3, 2025
@Abdomash Abdomash marked this pull request as ready for review March 4, 2025 23:39
Copy link
Member

@IsaDavRod IsaDavRod left a comment

Choose a reason for hiding this comment

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

I made a new component in the Figma for the feature list and fixed some spacing that was missing a variable. Please make the requested changes below:

@Abdomash Abdomash requested a review from IsaDavRod March 5, 2025 21:54
Copy link
Member

@IsaDavRod IsaDavRod left a comment

Choose a reason for hiding this comment

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

LGTM

@Samathingamajig Samathingamajig self-requested a review March 6, 2025 01:46
Copy link
Collaborator

@Samathingamajig Samathingamajig left a comment

Choose a reason for hiding this comment

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

will have more time to review later, but left an important comment

@Abdomash Abdomash requested a review from Samathingamajig March 8, 2025 00:59
@DereC4 DereC4 self-requested a review March 8, 2025 01:55
Copy link
Member

@DereC4 DereC4 left a comment

Choose a reason for hiding this comment

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

image

Looks good to me, and the version is now tracked using numbers, not a boolean. Other than that, the one thing left is what we're going to do with the graphic/preview; keeping it in the extension for everyone is not ideal right

@DereC4 DereC4 added the feature label Mar 8, 2025
@DereC4
Copy link
Member

DereC4 commented Mar 8, 2025

image
image

Copy link
Member

@IsaDavRod IsaDavRod left a comment

Choose a reason for hiding this comment

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

LGTM

@Samathingamajig Samathingamajig self-requested a review March 8, 2025 21:22
Copy link
Collaborator

@Samathingamajig Samathingamajig left a comment

Choose a reason for hiding this comment

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

LGTM!! great work!

@Samathingamajig Samathingamajig dismissed Razboy20’s stale review March 8, 2025 21:40

approved by elie

Sam — Today at 3:38 PM
thats what i think too just wanted to throw that out there
@elie did you want to review this again or you fine? Both of your requested changes were addressed
Elie — Today at 3:39 PM
Looks good 👍
Sam — Today at 3:39 PM
k
Elie — Today at 3:39 PM
(on mobile)

@Samathingamajig Samathingamajig merged commit f036d40 into Longhorn-Developers:main Mar 8, 2025
6 checks passed
@Samathingamajig
Copy link
Collaborator

sorry other elie for the ping, got unlucky with the copy/pasting having a name conflict

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a "What's New" prompt when the user opens calendar for the first time after an update
6 participants