Open Tandashi opened 1 year ago
Thanks for the reproducible example! This could be a quick fix, but I noticed I have a few other issues related to this.
Like the fact that get.currentRound()
should not be a thing anymore, because it's actually possible to have 2 rounds running at the same time: the final and the consolation final of a single elimination match.
And also in double elimination, where a round of the winner bracket can be played a the same time as a round of the loser bracket - although it's not implemented for double elimination yet, but it better be future proof.
Would it maybe be useful to implement it similar to get.currentMatches
but just for rounds?
Meaning get.currentRounds
returns an array of all the rounds that can be played. Then the consumer can also sort by number
if previous rounds want to be played first for example.
This would also work for both single_elimination
and double_elimination
.
I don't have the bandwidth to do it currently. PRs are welcome 😉
Will make some room for this later today. Feel free to assign this to me if you'd like 😎
I just implemented the quick fix for now due to time limitations as well :|
@Tandashi Can you maybe make a local patch on your repo for now?
With https://www.npmjs.com/package/patch-package for example.
Thanks for the PR but I won't have my computer for one week, I took some days off.
Currently I am running into the problem that
manager.get.currentRound
returns a round that only hasLocked
orCompleted
matches. This only happens if there areBYE
s involved.Probably similar to: https://github.com/Drarig29/brackets-manager.js/issues/172
Summary
Version:
1.6.3
Example