Skip to content

Conversation

@Ishuuu1213
Copy link
Contributor

No description provided.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Summary

This PR implements a dedicated challenges page feature that transitions CodeClip from a single-page application to a multi-page structure. The main changes include:

  1. New Challenges Page Integration: Updates the navigation in index.html to link to a dedicated pages/challenges.html file instead of using hash-based navigation, moving away from the single-page app pattern.

  2. Enhanced Challenge Execution Options: Adds "Run in JS Executor" buttons to all four challenge cards in pages/challenges.html, providing users with access to a more sophisticated multi-language code execution environment that uses the Judge0 API and supports 75+ programming languages, complementing the existing basic editor option.

  3. Leaderboard Reset: Clears all existing contributor data from LEADERBOARD.md, removing 36 contributors' achievements and replacing them with a "No contributors yet" placeholder.

The changes align with CodeClip's goal of providing comprehensive coding practice tools by offering users choice between a simple editor and a full-featured execution environment. The new challenges page structure allows for better organization of challenge-related functionality with individual challenge cards, filtering capabilities, and multiple interaction options.

Confidence score: 2/5

  • This PR has significant issues that will likely cause immediate problems in production, particularly with navigation and data loss.
  • The navigation path uses a repository-specific relative path that will break in most deployment scenarios, and the leaderboard reset removes valuable contributor data that conflicts with the automated regeneration system.
  • All files need attention, but index.html and LEADERBOARD.md require immediate fixes before merging.

3 files reviewed, 1 comment

Edit Code Review Bot Settings | Greptile

@Ishuuu1213
Copy link
Contributor Author

@adityai0 Sir it will close issue number #262

@adityai0 adityai0 merged commit a58931b into opensource-society:main Jul 27, 2025
1 check failed
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