Skip to content

Feat/sql history#23

Merged
ty-kim7 merged 3 commits into
mainfrom
feat/sql-history
Oct 14, 2025
Merged

Feat/sql history#23
ty-kim7 merged 3 commits into
mainfrom
feat/sql-history

Conversation

@ty-kim7

@ty-kim7 ty-kim7 commented Oct 14, 2025

Copy link
Copy Markdown

This pull request introduces a new feature for SQL challenges: users can now view their submission history via an API endpoint and a modal in the UI. It also adds security improvements to challenge previewing and enhances the user experience by preventing copying or selecting challenge descriptions. Additionally, several static asset files are updated, likely due to a rebuild after code changes.

SQL Challenge Submission History Feature:

  • Added a new API endpoint (/api/v1/challenges/<int:challenge_id>/sql-submissions) to fetch a user's submission history for SQL challenges, with authentication required and proper error handling.
  • Implemented a modal and supporting JavaScript (showSubmissionHistory) to display submission history in the challenge UI, including formatted dates and status indicators. [1] [2]

Security and Access Control Improvements:

  • Updated challenge preview logic to restrict previewing hidden challenges to admins only, aborting with a 403 error for unauthorized users.

User Experience Enhancements:

  • Prevented text selection, copying, and right-clicking in the challenge description panel to discourage content leakage or cheating.

Static Asset Updates:

  • Rebuilt and renamed static asset files (challenges.cd891a5f.js, clipboard.6907d1dc.js, index.86a4c4d6.js) to reflect dependency changes and new code for the above features. [1] [2] [3]

Miscellaneous:

  • Commented out the linting workflow trigger in .github/workflows/lint.yml, possibly for testing or to disable CI linting temporarily.

@ty-kim7
ty-kim7 merged commit f86f6f7 into main Oct 14, 2025
@ty-kim7
ty-kim7 deleted the feat/sql-history branch October 17, 2025 15:10
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.

1 participant