`manager.get.currentRound` returns outdated Round with all Matches being `Locked` or `Completed` about brackets-manager.js HOT 6 OPEN

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.

from brackets-manager.js.

I don't have the bandwidth to do it currently. PRs are welcome 😉

from brackets-manager.js.

Will make some room for this later today. Feel free to assign this to me if you'd like 😎

from brackets-manager.js.

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

from brackets-manager.js.

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.

from brackets-manager.js.

I just implemented the quick fix for now due to time limitations as well :|

from brackets-manager.js.