Skip to content

Commit

Permalink
Fix bug where assigning a table and first breaker to the first fixtur…
Browse files Browse the repository at this point in the history
…e in a round robin resulted in that fixture being de-prioritised when the second fixture was assigned to a table
  • Loading branch information
phrasmotica committed Jul 6, 2024
1 parent c0633c9 commit d41f070
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
4 changes: 4 additions & 0 deletions src/components/play/FixtureCard.vue
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,10 @@ const fixtureCardClass = (fixture: Fixture) => [
v-if="fixture"
class="border-round-md border-1"
:class="fixtureCardClass(fixture)">
<div v-if="uiStore.isDebugMode" class="text-center">
{{ fixture.id }}
</div>

<div v-if="table && !fixture.finishTime" class="text-center">
<TableBadge :table="table" />
</div>
Expand Down
4 changes: 3 additions & 1 deletion src/composables/usePhase.ts
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,9 @@ export const usePhase = (p: Phase | null) => {
const status = getFixtureStatus(fixture)

const validStatuses = [
FixtureStatus.ReadyToStart,
// TODO: FixtureStatus.ReadyToStart, but only if this fixture would
// be brought forwards in time. Probably needs a check in the place
// that's calling this function...
FixtureStatus.WaitingForAssignment,
]

Expand Down

0 comments on commit d41f070

Please sign in to comment.