Skip to content

feat(admin): add spare trigger assign#41

Merged
asd-a merged 1 commit into
mainfrom
trigger-assign
May 25, 2025
Merged

feat(admin): add spare trigger assign#41
asd-a merged 1 commit into
mainfrom
trigger-assign

Conversation

@Early0v0
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 25, 2025 15:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds the spare trigger assignment feature for admin users.

  • Introduces a new asynchronous function to call the spare_trigger_assign API endpoint with week information.
  • Adds a new UI segment that provides a button to initiate the assignment and displays the result message using the Message component.
Comments suppressed due to low confidence (1)

src/pages/Admin.tsx:498

  • [nitpick] Consider renaming the 'submit' function to something more descriptive like 'handleSpareTriggerAssign' to better reflect its purpose.
const submit = async () => {

Comment thread src/pages/Admin.tsx
@Early0v0 Early0v0 requested a review from CoderOJ May 25, 2025 15:41
@sonarqubecloud
Copy link
Copy Markdown

@asd-a asd-a merged commit 6434969 into main May 25, 2025
2 checks passed
@asd-a asd-a deleted the trigger-assign branch May 25, 2025 15:52
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