Skip to content
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

Merged
merged 27 commits into from
Oct 10, 2024
Merged

feat(match2): support hearthstone #4518

merged 27 commits into from
Oct 10, 2024

Conversation

Kanopedia
Copy link
Collaborator

@Kanopedia Kanopedia commented Aug 12, 2024

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

@hjpalpha
Copy link
Collaborator

fwiw from what i read on discord they have BR matches too ...

Copy link
Collaborator

@hjpalpha hjpalpha left a 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

@Rathoz Rathoz changed the title Create Match Summary for hearthstone feat(match2): support hearthstone Aug 12, 2024
@hjpalpha
Copy link
Collaborator

hjpalpha commented Sep 3, 2024

needs testing! @Kanopedia

@Rathoz
Copy link
Collaborator

Rathoz commented Sep 9, 2024

#4650 adjustment needed

@Rathoz
Copy link
Collaborator

Rathoz commented Oct 7, 2024

Update Info, missing Match Legacy

@Rathoz Rathoz marked this pull request as draft October 7, 2024 09:00
@Rathoz Rathoz marked this pull request as ready for review October 10, 2024 07:11
Kanopedia added a commit that referenced this pull request Oct 10, 2024
Rathoz pushed a commit that referenced this pull request Oct 10, 2024
Rathoz added a commit that referenced this pull request Oct 10, 2024
* 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]>
hjpalpha and others added 12 commits October 10, 2024 15:29
* 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]>
@Rathoz Rathoz force-pushed the Kanopedia-patch-1 branch from 7fcb8ad to 4af5add Compare October 10, 2024 13:30
Copy link
Collaborator

@Rathoz Rathoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Rathoz Rathoz merged commit cea3f5e into main Oct 10, 2024
5 checks passed
@Rathoz Rathoz deleted the Kanopedia-patch-1 branch October 10, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants