Skip to content

[FOSSOVERFLOW-25] Fixed an issue where the search bar kept re-rendering on each input. #56

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.

Sign up for GitHub

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

Open
alSN0W wants to merge 36 commits into
base: main
Choose a base branch
from
Open

[FOSSOVERFLOW-25] Fixed an issue where the search bar kept re-rendering on each input. #56

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
8d13a9c
Added api endpoints for voting on reviews.
alSN0W Feb 9, 2026
268dca8
fixed the faulty api endpoints, verified with postman
alSN0W Feb 10, 2026
004f15d
Merge pull request #1 from alSN0W/review-engagement
alSN0W Feb 10, 2026
e66517d
added vote button
alSN0W Feb 12, 2026
3966142
Merge pull request #2 from alSN0W/review-engagement
alSN0W Feb 12, 2026
888a1b9
fixed displaying the user votes incorrectly issue
alSN0W Feb 12, 2026
74dd74a
Merge pull request #3 from alSN0W/review-engagement
alSN0W Feb 12, 2026
e2659ed
Styled Voting buttons and updated migration files
alSN0W Feb 14, 2026
4348aba
Merge pull request #4 from alSN0W/review-engagement
alSN0W Feb 14, 2026
b37d929
removed useVote as it's useless for now
alSN0W Feb 15, 2026
4b35ddf
Merge pull request #5 from alSN0W/review-engagement
alSN0W Feb 15, 2026
f2ac58d
added pagination type and utils
alSN0W Feb 15, 2026
22fb266
Merge pull request #6 from alSN0W/review-engagement
alSN0W Feb 15, 2026
9f6ce6c
Revert "added pagination type and utils"
alSN0W Feb 17, 2026
c218637
Merge pull request #7 from alSN0W/review-engagement
alSN0W Feb 17, 2026
3c77c72
made changes recommended by code-rabbit
alSN0W Feb 17, 2026
2adff6a
Merge pull request #8 from alSN0W/review-engagement
alSN0W Feb 17, 2026
d461b81
Update src/lib/withPagination.ts
alSN0W Feb 18, 2026
4cf96ac
Update src/components/common/VoteButton.tsx
alSN0W Feb 18, 2026
46cb5e1
Update src/lib/pagination.ts
alSN0W Feb 18, 2026
0dfd0fb
Update src/components/common/VoteButton.tsx
alSN0W Feb 18, 2026
f8da0eb
some code-rabbit changes
alSN0W Feb 18, 2026
40361b9
Implemented pagination UI for reviews
alSN0W Feb 21, 2026
e477d5e
Merge pull request #9 from alSN0W/review-engagement
alSN0W Feb 21, 2026
6e6e38a
documentation changes
alSN0W Feb 21, 2026
0210f87
Merge pull request #10 from alSN0W/review-engagement
alSN0W Feb 21, 2026
cf07378
removed all the useless routes and functions
alSN0W Feb 23, 2026
51ef5a2
Merge branch 'main' into review-engagement
alSN0W Feb 23, 2026
23665f3
Merge pull request #11 from alSN0W/review-engagement
alSN0W Feb 23, 2026
d5cf99f
Revert "removed all the useless routes and functions"
alSN0W Feb 24, 2026
8d2552e
Merge pull request #12 from alSN0W/revert-11-review-engagement
alSN0W Feb 24, 2026
ff3dd90
Revert "Revert "removed all the useless routes and functions""
alSN0W Feb 24, 2026
b7da68d
Merge pull request #13 from alSN0W/revert-12-revert-11-review-engagement
alSN0W Feb 24, 2026
3889f33
final checks
alSN0W Feb 25, 2026
f58f3a6
updated migration with rls for votes table
alSN0W Feb 26, 2026
3deea90
fixed searchbar bug and implemented instant search results
alSN0W Mar 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion next-env.d.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
/// <reference types="next" />
/// <reference types="next/image-types/global" />
/// <reference types="next/navigation-types/compat/navigation" />
import "./.next/dev/types/routes.d.ts";

// NOTE: This file should not be edited
// see https://nextjs.org/docs/basic-features/typescript for more information.
// see https://nextjs.org/docs/app/api-reference/config/typescript for more information.
Comment on lines +4 to +7
Copy link

@coderabbitai coderabbitai bot Mar 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🌐 Web query:

Next.js 16 typed routes configuration

💡 Result:

To enable typed routes (statically typed links) in Next.js 16, turn on the stable typedRoutes flag in your next.config.* (don’t use experimental.typedRoutes anymore). [1]

// next.config.ts
import type { NextConfig } from 'next'

const nextConfig: NextConfig = {
  typedRoutes: true,
}

export default nextConfig

Requirements / what you get:

  • Your project must be using TypeScript. [1]
  • On next dev / next build, Next.js generates route/link type definitions under .next/types, which TypeScript uses to type-check Link href (and related navigation usage). [2]

Optionally, you can use the Route type from next:

import type { Route } from 'next'

const href: Route = '/dashboard'

[2]

Sources: [1] [2]

🏁 Script executed:

# Check the Next.js version in package.json
fd package.json -x cat {} | grep -A 2 -B 2 "next"

Repository: OpenLake/RateMyCourse

Length of output: 745

🏁 Script executed:

# Find and read the next-env.d.ts file
fd next-env.d.ts -x cat -n {}

Repository: OpenLake/RateMyCourse

Length of output: 430

🏁 Script executed:

# Search for how Next.js 16 generates and structures the typed routes file
# Let's look for any .next directory structure hints or related configuration
fd -t d ".next" -x echo {} 2>/dev/null | head -20

Repository: OpenLake/RateMyCourse

Length of output: 47

🏁 Script executed:

# Search the codebase for any references to the .next directory, next.config files, or typedRoutes configuration
rg -i "typedroutes|typed.*routes" -A 2 -B 2 --type ts --type js --type json

Repository: OpenLake/RateMyCourse

Length of output: 47

🏁 Script executed:

# Check if next.config file exists and its content
fd "next.config" -x cat -n {}

Repository: OpenLake/RateMyCourse

Length of output: 573

Remove or enable the typed routes import based on project needs.

This file is auto-generated by Next.js and should not be manually edited. Line 4 imports from ./.next/dev/types/routes.d.ts, which will only be generated if you enable typed routes. Either remove this import if typed routes are not needed, or enable them in next.config.js with typedRoutes: true (not experimental.typedRoutes, which is no longer used in Next.js 16). Once enabled, Next.js will regenerate this file automatically on each build.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@next-env.d.ts` around lines 4 - 7, The auto-generated import of
"./.next/dev/types/routes.d.ts" in next-env.d.ts should either be removed or
enabled via Next.js typed routes; either delete that import line from
next-env.d.ts if you don't need typed routes, or enable typed routes by adding
typedRoutes: true to your next.config.js (do not use experimental.typedRoutes).
After enabling, rebuild so Next.js regenerates this file automatically.

2 changes: 2 additions & 0 deletions next.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,13 @@ const nextConfig = {
eslint: {
ignoreDuringBuilds: true,
},
turbopack: {},
images: { unoptimized: true },
webpack: (config) => {
config.ignoreWarnings = [
{
module: /@supabase\/realtime-js/,

},
];
return config;
Expand Down
Loading