Skip to content

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

Merged
jackshim415 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
jackshim415 merged 8 commits into
mainfrom
jackshim415/lvt-40-warn-user-if-manually-entered-match-info-doesnt-match

Conversation

@jackshim415

Copy link
Copy Markdown
Contributor

No description provided.

@jackshim415
jackshim415 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
@jackshim415
jackshim415 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
@jackshim415
jackshim415 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
@jackshim415
jackshim415 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