-
Notifications
You must be signed in to change notification settings - Fork 2
Fix/companyfunctions – Improve Company Analytics Functions & Stability** #323
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
…URITY.md - Expand README with comprehensive platform highlights and value proposition - Reorganize features section with detailed descriptions of core capabilities - Update intellectual property notice with clearer GitHub organization information - Add emphasis on multi-company marketplace, analytics, and event management features - Remove outdated SECURITY.md template file - Enhance company events management page with event deletion functionality - Add delete event button with confirmation dialog to prevent accidental removal - Implement API integration for event deletion with proper error handling and user feedback - Add loading state indicator during event deletion process - Consolidate event management UI with edit, view, and delete actions
…any info - Reorganize edit event page with improved dark mode support and full-height container - Move delete button to danger zone section at bottom of page with descriptive warning - Simplify back button styling and positioning for better UX - Add collapsible company information section to reduce visual clutter - Implement Collapsible UI component for expandable/collapsible sections - Update EventForm with collapsible company info and improved dark mode styling - Add ChevronDown icon for collapsible state indication - Improve visual hierarchy and spacing throughout the edit event interface - Enhance dark mode compatibility with updated background and border colors
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. WalkthroughDocumentation updates including README reframing and SECURITY.md removal. Event management pages enhanced with deletion workflows and confirmation dialogs. Event edit page restructured with delete moved to danger zone. EventForm adds collapsible company information section. New collapsible UI component introduced. Radix UI collapsible dependency added. Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant EventsPage
participant AlertDialog
participant API as /api/events/{slug}
participant Toast
participant EventsList
User->>EventsPage: Click delete button
EventsPage->>AlertDialog: Show confirmation dialog
User->>AlertDialog: Confirm deletion
AlertDialog->>EventsPage: Trigger handleDeleteEvent
EventsPage->>EventsPage: Set deletingEventSlug, disable button
EventsPage->>API: DELETE request
rect rgb(100, 200, 100)
API-->>EventsPage: Success response
EventsPage->>Toast: Show success notification
end
alt Error occurs
rect rgb(200, 100, 100)
API-->>EventsPage: Error response
EventsPage->>Toast: Show error notification
end
end
EventsPage->>EventsList: Refresh events list
EventsPage->>EventsPage: Clear deletingEventSlug
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (8)
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 pull request introduces important fixes and improvements to the company-related backend functions, ensuring accurate analytics tracking, better stability, and alignment with the latest schema updates.
✅ What’s Included
increment_company_analyticsto prevent incorrect field updates.🛠️ Changes Made
🚀 Why This Matters
These fixes ensure:
🔍 Testing
All affected functions were manually tested with:
No breaking changes observed.
🔗 Closing
This PR fixes the underlying company analytics issues and improves internal documentation and UI consistency.
Authored by: @akshay0611
Summary by CodeRabbit
New Features
Documentation