-
Notifications
You must be signed in to change notification settings - Fork 70
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
feat: implement a What's New prompt #539
Conversation
Don't we already have this |
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.
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:
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.
LGTM
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.
will have more time to review later, but left an important comment
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.
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.
LGTM
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.
LGTM!! great work!
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)
sorry other elie for the ping, got unlucky with the copy/pasting having a name conflict |
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
TODO:
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 changingDialog.tsx
)This change is