Skip to content

live view: show upcoming matches (#58)#80

Merged
davidohnee merged 4 commits intomainfrom
58-live-view-show-upcoming-matches
Mar 3, 2026
Merged

live view: show upcoming matches (#58)#80
davidohnee merged 4 commits intomainfrom
58-live-view-show-upcoming-matches

Conversation

@davidohnee
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 3, 2026 16:29
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 3, 2026

Deploying bracketeer with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9edc837
Status: ✅  Deploy successful!
Preview URL: https://cb237a27.bracketeer-3cf.pages.dev
Branch Preview URL: https://58-live-view-show-upcoming-m.bracketeer-3cf.pages.dev

View logs

Copy link
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 an "Upcoming Matches" view to the live tournament view during breaks between rounds. When a round finishes and the tournament is in a break state, instead of only showing recent results, users can now toggle between a "Results" tab and an "Upcoming" tab via a TabSelector. Additionally, the TabSelector component gets a new optional showNavigation prop (defaulting to hidden), and a pushToRemote function is added to the tournaments store. Several dependency bumps are also included.

Changes:

  • TournamentLive.vue: Added a TabSelector between rounds to switch between "Results" and "Upcoming" match views; resets to "Results" view on adjustAndSkip; uses the new store push method
  • TabSelector.vue: Made navigation arrows optional via new showNavigation prop (hidden by default); tabs now center-align when navigation is hidden
  • src/stores/tournaments.ts: Added pushToRemote function exposed as push that conditionally shares a tournament if it has a remote target
  • package.json / package-lock.json: Bumped multiple dependencies including vue-router v4 → v5, vue v3.5.22 → v3.5.29, and various dev tooling

Reviewed changes

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

Show a summary per file
File Description
src/components/TournamentLive.vue Adds TabSelector UI for switching between Results/Upcoming match views during break state
src/components/TabSelector.vue Makes navigation arrows optional via showNavigation prop
src/stores/tournaments.ts Adds conditional pushToRemote helper exposed as push on the store
src/assets/css/design/colors.css Slight background color adjustment
package.json Dependency version bumps
package-lock.json Lock file update to match new dependency versions

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@davidohnee davidohnee merged commit 8358296 into main Mar 3, 2026
8 checks passed
@davidohnee davidohnee deleted the 58-live-view-show-upcoming-matches branch March 3, 2026 16:53
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