Skip to content

LVT-40: Warn user if manually entered match info doesn't match schedule#53

Merged
jackattack-4 merged 8 commits into
mainfrom
jackshim415/lvt-40-warn-user-if-manually-entered-match-info-doesnt-match
Jul 8, 2026
Merged

LVT-40: Warn user if manually entered match info doesn't match schedule#53
jackattack-4 merged 8 commits into
mainfrom
jackshim415/lvt-40-warn-user-if-manually-entered-match-info-doesnt-match

Conversation

@jackattack-4

Copy link
Copy Markdown
Contributor

No description provided.

@jackattack-4 jackattack-4 requested a review from Copilot July 3, 2026 17:14
@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 3, 2026 17:14 Destroyed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a manager endpoint intended to validate whether a manually-entered match reference (tournament/team/match number/type) corresponds to an existing scheduled match, supporting UI warnings when user input doesn’t match the schedule.

Changes:

  • Added a new Manager API route (GET /v1/manager/checkmatch) and OpenAPI documentation for it.
  • Implemented checkMatchExists handler that ensures tournament matches are loaded and queries teamMatchData for the requested match row.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
src/routes/manager/manager.routes.ts Registers OpenAPI spec for /checkmatch and wires the Express route into the manager router.
src/handler/manager/checkMatchExists.ts New handler to validate query params, load tournament matches, and look up teamMatchData for the requested match.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/routes/manager/manager.routes.ts
Comment thread src/routes/manager/manager.routes.ts
Comment thread src/routes/manager/manager.routes.ts
Comment thread src/handler/manager/checkMatchExists.ts Outdated
Comment thread src/handler/manager/checkMatchExists.ts Outdated
Comment thread src/handler/manager/checkMatchExists.ts
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 3, 2026 17:21 Destroyed
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 3, 2026 17:21 Destroyed
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 3, 2026 17:22 Destroyed
@railway-app

railway-app Bot commented Jul 3, 2026

Copy link
Copy Markdown

🚅 Deployed to the lovat-server-pr-53 environment in lovat

Service Status Web Updated (UTC)
lovat-server ✅ Success (View Logs) Web Jul 3, 2026 at 6:44 pm

@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 3, 2026 18:40 Destroyed
@jackattack-4 jackattack-4 requested a review from Copilot July 6, 2026 00:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

Comment thread src/routes/manager/manager.routes.ts
Comment thread src/routes/manager/manager.routes.ts
Comment thread src/handler/manager/checkMatchExists.ts
Comment thread src/handler/manager/checkMatchExists.ts Outdated
Comment thread src/handler/manager/checkMatchExists.ts Outdated
Comment thread src/routes/manager/manager.routes.ts Outdated
@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 8, 2026 03:24 Destroyed
@jackattack-4 jackattack-4 requested a review from Copilot July 8, 2026 03:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

Comment thread src/routes/manager/manager.routes.ts
Comment thread src/routes/manager/manager.routes.ts Outdated
Comment thread src/routes/manager/manager.routes.ts
Comment thread src/routes/manager/manager.routes.ts
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 8, 2026 03:30 Destroyed
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@railway-app railway-app Bot temporarily deployed to lovat / lovat-server-pr-53 July 8, 2026 03:30 Destroyed
@jackattack-4 jackattack-4 merged commit b29ee07 into main Jul 8, 2026
4 checks passed
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