Skip to content

Add new subject form#50

Open
nquynh06 wants to merge 1 commit intomainfrom
feat/add-subject-form
Open

Add new subject form#50
nquynh06 wants to merge 1 commit intomainfrom
feat/add-subject-form

Conversation

@nquynh06
Copy link
Contributor

@nquynh06 nquynh06 commented Feb 8, 2026

No description provided.

@snyk-io
Copy link

snyk-io bot commented Feb 8, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@thu4n
Copy link
Contributor

thu4n commented Feb 8, 2026

  • Add description text box to add more info about the new course
  • Show them the PR URL after successful submission
  • If they add it directly to their grade table, let them know that the data exist temporarily and will be lost after cache deletion or new website update
  • Use GitHub Apps to create PR instead of using PAT
  • Add ceiling for course credits
  • User must login to GitHub to submit the form

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.

2 participants