feat(PrizePool): add playerShare & clubShare support.#7284
Conversation
c4861c6 to
3537a31
Compare
|
collapsibility might not be needed. consensus appears to be about 50/50 |
|
i think you also want to adjust TeamParticipant (to use the playershare there for calculation of the correct indiv prize money) for this to take the full effect you want |
Co-authored-by: SyntacticSalt <mbergent@uni-bremen.de>
also ensure that when set it does not duplicate club share columns
There was a problem hiding this comment.
I am quite opposed to putting this in Commons at all, this looks very counterstrike-specific (maybe dota2 too, sure, but that's the end of the list anyway) and thus implies unnecessary runtime overhead on all other wikis
if this really should go to Commons, then imo it should be placed under /Valve variant or something (similar to how sc and sc2 share some custom modules)
as i understand it it is relevant for all team based wikis that are present at ewc |
tbf I was partially wrong here. club share exists at EWC, but it appears you already handle EWC at a esports wiki level (didnt know this existed) It isnt Valve specific though as theoretically operators who use a club share system i.e BLAST, ESL etc could use it in their other titles. Just a majority of other titles are franchise systems where it wouldnt apply. |
|
Fwiw I doubt there is any drastic runtime overhead with this |
Also would this not apply to F1? Its obviously different terminology but the same principle no? Edit: I guess not actually |
d5139b9 to
c536845
Compare
|
@Rathoz if you have a bit of spare time, your opinion on this would be appreciated |
* try to resolve comment * nil catch * chore: update visual snapshots * lets try this * chore: update visual snapshots * extension ate a char again ...
ElectricalBoy
left a comment
There was a problem hiding this comment.
still not a fan of this PR, but whatever
SlothyMan
left a comment
There was a problem hiding this comment.
Its lgtm (from noob perspective)
Rathoz
left a comment
There was a problem hiding this comment.
How does this work with non-usd pricepools?
it works, it just isnt pretty https://liquipedia.net/counterstrike/User:MischiefCS/PGL/2025/Astana Not much work was done for it as it doesnt ever really apply in CS, and all clubShares will have a specific dollar value. Ill admit it doesnt work well from a wiki perspective though. The proposed solution in editor-announcements is far better |
|
likely superseded by #7728 |

Summary
Primarily driven by the cs wiki due to how nearly all Tier 1 events have incorrect prize data, but I believe this may also be appropriate for dota2.
In a majority of Tier 1 events, they now introduced a club share split. For how the cs wiki works (and I believe all other wikis), clubshare does not really need to be tracked in itself. At a team level we have always attributed the full prizepool, even if the players were getting part or some.
Our only difference is how we attribute the players cut of the prizepool. Thus far we have only represente the "player share" in tier 1 events as we dont want to inflate their earnings. This PR aims to interject the individualprizemoney path such that it reads from the playershare only, and the event & teams can read the true data which we will be able to add to the wiki.
This has been one of the primary requests fromc s contributors for the past 15 odd months,
How did you test this change?
Deployed under /dev/MischiefClub to Commons and counterstrike
Test page at: https://liquipedia.net/counterstrike/User:MischiefCS/PGL/2025/Astana
Im not sure how to test players receiving the earnings when its such a large /dev/ package as this.
This also doesnt look great on mobile, its been discussed ideally we would have a collapsible js function to colapse the two shares, not sure how to achieve this.
Likewise, its been suggested that ideally would have an explainer / note higlight for the two shares. Again not too sure how to achieve this.