-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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/notifications v2 #19162
feat/notifications v2 #19162
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Ignored Deployments
|
Updates to Preview Branch (feat/notifications-v2) ↗︎
Tasks are run on every commit but only new migration files are pushed.
Connecting to this Preview Branch
Branching Preview Branches by Supabase. |
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!
Couple of small nits:
...dio/components/layouts/ProjectLayout/LayoutHeader/NotificationsPopoverV2/NotificationRow.tsx
Outdated
Show resolved
Hide resolved
...omponents/layouts/ProjectLayout/LayoutHeader/NotificationsPopoverV2/NotificationsPopover.tsx
Show resolved
Hide resolved
...omponents/layouts/ProjectLayout/LayoutHeader/NotificationsPopoverV2/NotificationsPopover.tsx
Outdated
Show resolved
Hide resolved
* Midway implementation notifications v2 * Some updates to Tab component * Midway implementation notifications v2 again * implement infinite scrolling and loading for notification sv2 * Add button actions * Implement archive notifications * Add filter dropdown for notifications * Update date format * Small style fix * Update * Update API codegen * Add empt state * Add empty state for archived notifications * Fix type * Final changes to notifications v2 * Small style fix * Small fix * Only mark notifications that come into view as read * Update package lock
Example: