-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(match2): support hearthstone #4518
Conversation
fwiw from what i read on discord they have BR matches too ... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR title not fitting
components/match2/wikis/hearthstone/get_match_group_copy_paste_wiki.lua
Outdated
Show resolved
Hide resolved
needs testing! @Kanopedia |
#4650 adjustment needed |
Update Info, missing Match Legacy |
* Update Hearthstone Info Based on: #4518 (comment) * Update standard/info/wikis/hearthstone/info.lua Co-authored-by: Rikard Blixt <[email protected]> --------- Co-authored-by: Rikard Blixt <[email protected]>
* Update Hearthstone Info Based on: #4518 (comment) * Update standard/info/wikis/hearthstone/info.lua Co-authored-by: Rikard Blixt <[email protected]> --------- Co-authored-by: Rikard Blixt <[email protected]>
* Create match_legacy for Hearthstone * Apply suggestions from code review Co-authored-by: hjpalpha <[email protected]> * Update components/match2/wikis/hearthstone/match_legacy.lua * Update components/match2/wikis/hearthstone/match_legacy.lua * Update components/match2/wikis/hearthstone/match_legacy.lua --------- Co-authored-by: Rikard Blixt <[email protected]> Co-authored-by: hjpalpha <[email protected]>
7fcb8ad
to
4af5add
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
depends on #4645
Summary
Adding a basic match summary for 1 vs 1 tournaments in hearthstone.
(This is excluding the rare team events where individual players would need to be listed e.g aoe style.)
How did you test this change?
https://liquipedia.net/hearthstone/User:Kanoodles/Bracket