Nest switch #362
Merged
Nest switch #362
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added a new README for the comment system detailing features, architecture, and usage. - Integrated socket.io for real-time comment updates and reactions. - Created hooks for managing comments, reactions, and real-time interactions. - Developed UI components for comment threads, moderation dashboard, and user interactions. - Updated project and API types to support new comment system functionalities. - Implemented validation and content filtering for comments to enhance moderation capabilities.
…imizations - Updated project components to utilize new CrowdfundingProject type for better type safety. - Enhanced profile components with public views and improved data handling. - Introduced new hooks for managing follow statistics and user interactions. - Refactored various components to streamline data flow and improve performance. - Removed deprecated components and optimized existing ones for better maintainability. - Added new utility functions for handling user activities and project interactions.
- Upgraded Next.js and its associated packages in package.json and package-lock.json to version 16.0.10 for improved features and performance. - Ensured compatibility across all relevant modules by synchronizing version updates.
- Changed socket connection URLs in useCommentRealtime, useVoteRealtime, and useSocket hooks to utilize NEXT_PUBLIC_BETTER_AUTH_URL for improved configuration management.
- Introduced a new CheckEmail component to inform users about the verification email sent to their address. - Updated SignupForm to redirect users to the CheckEmail page after sending the verification email. - Refactored AuthPage to streamline mode handling and state synchronization with query parameters. - Removed unused state management in SignupForm for improved clarity and performance.
- Added a new AuthLayoutWrapper component for the accept invitation page, providing a consistent layout with animated transitions. - Enhanced the AcceptInvitationPage component to include a loading spinner and improved user feedback during the invitation acceptance process. - Updated various organization-related components to use the new 'id' field instead of '_id' for better consistency across the application. - Refactored organization-related API calls and state management to align with the new data structure.
- Replaced instances of '_id' with 'id' across various components and hooks for consistency. - Updated the Hackathon API and related types to reflect the new data structure. - Simplified resource handling in Hackathon components by removing unnecessary nesting. - Enhanced the organization-related components to utilize the new 'organizationId' field. - Cleaned up unused code and improved overall readability in several files.
- Introduced the HackathonParticipants component to display participant details. - Updated HackathonPage to integrate participant data and improve tab management. - Refactored participant data handling in useParticipants hook for better type safety and clarity. - Enhanced loading states and error handling in various components. - Cleaned up unused code and improved overall readability in related files.
- Updated CreateSubmissionModal to include participationType and set default values. - Modified submission handling in useSubmission hook to accommodate new data structure. - Removed deprecated discussions API and related hooks, transitioning to a more streamlined comment system. - Updated CommentItem component to reflect changes in comment structure and user data. - Adjusted API calls in lib/api to align with new backend requirements for submissions and comments. - Enhanced type definitions for comments and submissions to improve type safety and clarity. - Cleaned up unused code and comments related to discussions, focusing on the new comment structure.
…ions - Updated default Open Graph image URLs to use absolute paths. - Introduced utility functions for generating absolute image URLs, slugs, and truncating descriptions for SEO. - Enhanced page and blog metadata generation with additional fields for Open Graph, Twitter Card, and structured data. - Added JSON-LD structured data generation for blog posts and breadcrumbs. - Updated blog post type definitions to reflect changes in metadata structure. fix(organization): Remove debug trace from OrganizationProvider - Removed console.trace statement for organization data to clean up logs. refactor(blog): Update blog post structure and request types - Changed 'content' and 'image' fields to 'description' and 'coverImage' in BlogPost interface. - Added new fields for categories and updated request types to include filtering options for blog posts.
… navigation and data fetching
- Implemented participants API with registration, submission, and voting functionalities. - Created resources API to fetch hackathon resources. - Added rewards API for assigning ranks and managing escrow details. - Developed teams API for team recruitment posts and management. - Refactored hackathon types into core, draft, participant, judging, rewards, and team modules for better organization and maintainability. - Enhanced upload service to support multiple tags.
…g and loading state
…rTeam, and Blog components
… metadata generation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR implements a comprehensive "nest switch" refactoring that updates the codebase to use a new type system and API structure. The changes primarily focus on migrating from legacy type definitions to newer, more specific types, and updating component structures to align with Better Auth integration and improved data models.
Key Changes:
- Migration from generic
Projecttypes to specificCrowdfundingProjecttype - Integration of Better Auth for organization member management
- Updates to user profile types and data structures
- Refactoring of notification system to use WebSocket-based approach
Reviewed changes
Copilot reviewed 152 out of 289 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| components/profile/useProjectFilters.ts | Updated to use CrowdfundingProject type and removed tab-based filtering logic |
| components/profile/update/Settings.tsx | New settings component with form validation and improved state management |
| components/profile/update/Profile.tsx | New profile update component with avatar upload and validation |
| components/profile/UserStats.tsx | Removed click handlers for followers/following sections |
| components/profile/ProjectsTab*.tsx | Updated project display components with new type system |
| components/profile/ProfileHeader*.tsx | Refactored profile headers to use new user type structure |
| components/profile/ActivityFeed*.tsx | Updated activity feed to use new activity type structure |
| components/organization/* | Updated organization components for Better Auth integration |
| components/notifications/* | Migrated to WebSocket-based notifications system |
| components/landing-page/* | Updated landing page components with new type definitions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 'group relative flex items-center gap-3 rounded-xl px-3 py-3 text-sm font-medium transition-all', | ||
| isActive | ||
| ? 'from-primary/10 text-primary shadow-primary/5 bg-gradient-to-r to-transparent shadow-lg' | ||
| ? 'from-primary/10 text-primary shadow-primary/5 bg-linear-to-r to-transparent shadow-lg' |
There was a problem hiding this comment.
Invalid CSS class name bg-linear-to-r should be bg-gradient-to-r to properly render the gradient background.
Collaborator
Author
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
components/organization/hackathons/details/HackathonSidebar.tsx
Outdated
Show resolved
Hide resolved
Comment on lines
+135
to
+136
| ? 'border-zinc-700/50 bg-linear-to-br from-zinc-950 to-black opacity-60' | ||
| : 'border-zinc-800 bg-linear-to-br from-zinc-900 to-black hover:border-zinc-700' |
There was a problem hiding this comment.
Invalid CSS class name bg-linear-to-br should be bg-gradient-to-br to properly render the gradient background.
Collaborator
Author
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@0xdevcollins I've opened a new pull request, #363, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@0xdevcollins I've opened a new pull request, #364, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@0xdevcollins I've opened a new pull request, #365, to work on those changes. Once the pull request is ready, I'll request review from you. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.