-
Notifications
You must be signed in to change notification settings - Fork 2
Feat/multicompanylevel – Enhance event handling, usage tracking, and notifications #317
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
Conversation
…ttings - Add useSubscription hook to track event usage for current month - Update event creation service to set default approval_status to 'draft' - Modify company settings page to display dynamic event usage with limit warning - Add visual indicator for exceeding monthly event creation limit - Improve event usage tracking and display with more granular information
…ts page - Capitalize first letter of event status in admin events table - Update admin events API to filter out draft events - Modify event query to only show events with specific approval statuses - Enhance readability of event status display in table view
- Create new notifications page with dynamic filtering and interaction - Implement tabs for viewing all and unread notifications - Add notification statistics and summary cards - Support marking individual and all notifications as read - Include notification deletion functionality - Enhance UI with responsive design and intuitive actions - Integrate with existing useNotifications hook for data management - Add visual indicators for notification status and type - Implement time-based notification formatting
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. WalkthroughThe pull request refactors event status handling from pending to draft states, introduces a new notifications page with filtering and actions, integrates subscription usage data into the dashboard, and removes legacy database utility scripts. Multiple components across events, notifications, and dashboard features are updated cohesively. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant NotificationsPage
participant useNotifications
participant API
User->>NotificationsPage: Visit /protected/notifications
NotificationsPage->>useNotifications: Fetch notifications
useNotifications->>API: GET /api/notifications
API-->>useNotifications: Return notifications list
useNotifications-->>NotificationsPage: notifications[], loading, unread count
NotificationsPage->>NotificationsPage: Render cards with filter tabs
alt Notification Card Action
User->>NotificationsPage: Click action (open/read/delete)
NotificationsPage->>API: Update notification state
API-->>NotificationsPage: Success
NotificationsPage->>NotificationsPage: Refresh display
end
alt Mark All as Read
User->>NotificationsPage: Click "Mark all as read"
NotificationsPage->>API: POST mark all as read
API-->>NotificationsPage: Success
NotificationsPage->>useNotifications: Refetch notifications
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (11)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This PR introduces multi-company–level improvements across events and notifications, strengthening both backend logic and admin visibility.
🧩 Key Updates
Events Module
Notifications System
⚙️ Technical Improvements
🧪 Testing
🧍♂️ Review Notes
/admin/moderation.🔗 Closes
Closes internal tracking issues on event duplication and multi-company event management.
Authored by: @akshay0611
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
Chores