Skip to content

Conversation

@marcodejongh
Copy link
Owner

  • Created database query function to find climbs that use all holds of current climb plus additional holds
  • Implemented expandable Similar Climbs section above beta videos
  • Added API endpoint at /api/internal/similar-climbs for fetching similar climbs
  • Displays similar climbs in a responsive grid with climb cards
  • Helps users discover bigger versions of climbs created for larger board sizes

- Created database query function to find climbs that use all holds of current climb plus additional holds
- Implemented expandable Similar Climbs section above beta videos
- Added API endpoint at /api/internal/similar-climbs for fetching similar climbs
- Displays similar climbs in a responsive grid with climb cards
- Helps users discover bigger versions of climbs created for larger board sizes
@vercel
Copy link

vercel bot commented Nov 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
boardsesh Ready Ready Preview Comment Nov 9, 2025 8:22am

- Remove unused imports (and, inArray, ne) from similar-climbs.ts
- Add SimilarClimbRow interface to replace 'any' type
- Fix React Hook useEffect dependencies with useCallback and useRef
- Prevent duplicate API calls with hasFetchedRef
- Fetch similar climbs server-side in page component for better performance
- Pass data as props to SimilarClimbs component instead of client-side fetching
- Remove API endpoint /api/internal/similar-climbs (no longer needed)
- Set defaultActiveKey to expand section by default
- Simplify component by removing loading/error states and useEffect hooks
- Follows Next.js 15 server-side rendering best practices
- Fetch available sizes to determine the smallest size (minimum ID)
- Only fetch similar climbs if not on the smallest size board
- Conditionally render SimilarClimbs component based on board size
- Improves performance by skipping unnecessary queries on small boards
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants