added the changes on the landing page#4
Conversation
shubh-Rocks
commented
May 5, 2026
shubh-Rocks
commented
|
Someone is attempting to deploy a commit to the harshmahto02-gmailcom's projects Team on Vercel. A member of the Team first needs to authorize it. |
harshchill
left a comment
harshchill
left a comment
There was a problem hiding this comment.
Can be merged , but the UI can be improved , you have only removed the silk component from the bg of hero section and are still fetching it idk why , the UI can be improved
harshchill
left a comment
harshchill
left a comment
There was a problem hiding this comment.
🤖 AI Code Review
Summary
Reviewed 2 file(s):
app/component/Navbar.jsapp/page.js
app/component/Navbar.js
🔵 SUGGESTION: The changes made to the file are mostly formatting changes, with no significant logic changes. However, the formatting changes are inconsistent, with some lines being wrapped to the next line and some not. It would be better to maintain a consistent formatting style throughout the file.
Specifically, lines 19-22, 44-47, 84-87, 110-115, 125-128, and 136-139 have been reformatted, but the rest of the file remains unchanged. It would be better to reformat the entire file to maintain consistency.
Additionally, there are no critical or warning issues found in the code. The changes do not introduce any bugs or security vulnerabilities.
✅ LGTM for the logic changes, but consider reformatting the entire file for consistency.
app/page.js
🔵 SUGGESTION: The code has inconsistent spacing and formatting. For example, lines 3-18 have inconsistent indentation and spacing between imports. It's better to keep the formatting consistent throughout the file.
🔵 SUGGESTION: The fetchRecentPapers function is not handling potential errors well. Consider adding more specific error handling instead of just logging the error (line 26).
🔵 SUGGESTION: The setStats function is called with an object that has a total property. Consider adding a check to ensure that data.total and data.papers.length are numbers before calling Number() on them (line 30).
🔵 SUGGESTION: The useEffect hook is missing a dependency array. This could cause the effect to run unnecessarily. Consider adding a dependency array to the useEffect hook (line 17).
🔵 SUGGESTION: The code has magic numbers (e.g., 4 in fetch('/api/papers?limit=4&offset=0')). Consider defining these numbers as constants to improve readability and maintainability (line 24).
🟡 WARNING: The isAuthenticated variable is not defined in the provided code snippet. Make sure it's defined and handled correctly to avoid potential logic errors (line 54).
🔵 SUGGESTION: The code has long lines (e.g., line 63). Consider breaking them up into multiple lines to improve readability.
🔵 SUGGESTION: The ContributionsPage import is not used in the provided code snippet. Consider removing unused imports to improve code quality (line 10).
Reviewed by your PR Agent 🤖
![nemesis-chill[bot]](https://avatars.githubusercontent.com/in/4036478?s=60&v=4){kind=small}
There was a problem hiding this comment.
Nemesis Review Report
Summary
🟢 Verdict: LGTM
LGTM — clean, focused, no surprises. Ship it.
📝 What This PR Does
This PR updates the landing page with requested changes, enhancing the user experience and system functionality. The modifications aim to improve the overall presentation and interaction of the landing page.
🏗️ Key Changes
- Landing page updated with new content and layout adjustments
- No significant architectural changes or dependencies added
🚨 Must Fix Before Merge
Nothing blocking. Good to go.
app/component/Navbar.js
✅ No issues found.
app/page.js
✅ No issues found.
Issued by Nemesis
2 participants
