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

Add pseudo game "CS:GO/CS2" to cs wiki #3337

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Add pseudo game "CS:GO/CS2" to cs wiki #3337

merged 1 commit into from
Nov 10, 2023

Conversation

iMarbot
Copy link
Collaborator

@iMarbot iMarbot commented Sep 29, 2023

Summary

Because ESL decided to move already ongoing tournaments from CS:GO to CS2 (😠), we now have to deal with this. I thought about adding multiple game support to the infobox but that would require changing the base infobox too and not just CS custom. Also, what would we even do for the tournament_game var?? Would have to have like a switch date or something and maybe multiple vars, and then have to add display support for that in other modules too... so yeah, nope.

Instead just making this not really real game for the purpose. Then will override game per-match or using vardefine somewhere on the pages.

How did you test this change?

/dev and then pushed to live also as needed ASAP.

image

image

@hjpalpha
Copy link
Collaborator

#fuckESL

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.

LGTM

@mbergen
Copy link
Collaborator

mbergen commented Oct 4, 2023

#fuckESL

Can't really blame them for not having a public release date to plan with...

@hjpalpha
Copy link
Collaborator

hjpalpha commented Oct 4, 2023

#fuckESL

Can't really blame them for not having a public release date to plan with...

well the comment might also be due to what BS they do in sc2 atm

@Rathoz
Copy link
Collaborator

Rathoz commented Oct 6, 2023

Hm, it adds a lot of complexity for a small, short term gain, is it really needed?

@iMarbot
Copy link
Collaborator Author

iMarbot commented Oct 6, 2023

Hm, it adds a lot of complexity for a small, short term gain, is it really needed?

Complexity in what way? Also, I wouldn't call it short term because this isn't just while a tournament is ongoing, this is to accurately record that the tournament is and was played on two different games.

@hjpalpha
Copy link
Collaborator

hjpalpha commented Nov 9, 2023

RFM imo
according to Ricci Alex said he is okay with it^^

@Rathoz Rathoz merged commit 6d344f9 into main Nov 10, 2023
2 checks passed
@Rathoz Rathoz deleted the csgocs2-game branch November 10, 2023 18:52
@iMarbot iMarbot added data Changes to data modules and removed c: standard labels Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data Changes to data modules g: counter-strike Counter-Strike
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants