-
Notifications
You must be signed in to change notification settings - Fork 0
Final Product Merge #2
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
…s which have not yet been made
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Pull Request Overview
This PR merges multiple changes across the codebase to update UI components and API endpoints in preparation for the final product release. Key changes include an updated footer with new links and styling, enhancements to editor components with refined content management in the ContentArea, and multiple updates to API routes and user forms for improved error handling and user feedback.
Reviewed Changes
Copilot reviewed 33 out of 36 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/components/Footer.tsx | Updated footer links to use Link components with revised styling. |
| src/components/Editor/EditTools.tsx | Added and refactored editor toolbar actions. |
| src/components/Editor/ContentArea.tsx | Introduced state update for editor HTML content. |
| src/components/DisclaimerWarning.tsx | Enhanced disclaimer with an additional link. |
| src/app/page.tsx | Modified homepage sections and updated hero content. |
| src/app/find-out-more/page.tsx | Added an informational page about the project. |
| src/app/app/page.tsx | Created a new app page with left menu and middle component. |
| src/app/api/auth/[...nextauth]/options.ts | Adjusted auth options to throw errors instead of returning null. |
| src/app/api/(user)/signup/route.ts | Updated the signup endpoint with refined responses and comments. |
| src/app/api/(Post)/getPost/route.ts | Implemented pagination and enhanced error handling for posts. |
| src/app/api/(Post)/createpost/route.ts | Added a new endpoint for creating posts. |
| src/app/(User)/signup/SignUpForm.tsx | Updated signup form with additional comments and form hints. |
| src/app/(User)/signin/SignInForm.tsx | Enhanced signin form with session checks and improved messaging. |
| src/app/(User)/forgotpassword/ForgotPasswordForm.tsx | Updated forgot password form to redirect upon success. |
| src/ZodSchema/UserSchema.ts | Revised Zod schemas with added comments. |
| next.config.mjs | Removed experimental serverActions configuration. |
Files not reviewed (3)
- components.json: Language not supported
- package.json: Language not supported
- prisma/schema.prisma: Language not supported
| <li><Link href="/about" className='text-decoration: hover:underline hover:text-blue-500'>Our Mission</Link></li> | ||
| <li><Link href="/contact" className='text-decoration: hover:underline hover:text-blue-500'>Contact Us</Link></li> |
Copilot
AI
Mar 28, 2025
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.
The className contains 'text-decoration:' which is not a valid Tailwind utility. Consider removing 'text-decoration:' to correctly apply hover styles.
| <li><Link href="/about" className='text-decoration: hover:underline hover:text-blue-500'>Our Mission</Link></li> | |
| <li><Link href="/contact" className='text-decoration: hover:underline hover:text-blue-500'>Contact Us</Link></li> | |
| <li><Link href="/about" className='hover:underline hover:text-blue-500'>Our Mission</Link></li> | |
| <li><Link href="/contact" className='hover:underline hover:text-blue-500'>Contact Us</Link></li> |
| const html = editor.getHTML() | ||
| setContent(html) | ||
|
|
Copilot
AI
Mar 28, 2025
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.
Calling setContent during render may trigger an infinite re-render loop; consider using a useEffect hook to update state based on the editor's HTML output.
| const html = editor.getHTML() | |
| setContent(html) | |
| React.useEffect(() => { | |
| if (editor) { | |
| const updateContent = () => { | |
| const html = editor.getHTML() | |
| setContent(html) | |
| } | |
| editor.on('update', updateContent) | |
| return () => { | |
| editor.off('update', updateContent) | |
| } | |
| } | |
| }, [editor]) |
| <p className='text-centre'>You are currently logged in to log in with another account.</p> | ||
| <p className='text-centre'>To continue please first logout.</p> |
Copilot
AI
Mar 28, 2025
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.
The Tailwind CSS class 'text-centre' is misspelled; it should be 'text-center'.
| <p className='text-centre'>You are currently logged in to log in with another account.</p> | |
| <p className='text-centre'>To continue please first logout.</p> | |
| <p className='text-center'>You are currently logged in to log in with another account.</p> | |
| <p className='text-center'>To continue please first logout.</p> |
Doesn't live up to expecations but time is a thing.