Skip to content

Conversation

@hjpalpha
Copy link
Collaborator

@hjpalpha hjpalpha commented Dec 24, 2025

Summary

As per requests on discord add the option to exclude some tournaments from the Tournaments Ticker on main page.
This can be achieved by setting |hide_on_tournaments_ticker=true in the infobox of the according event.

start of discord conversation: https://discord.com/channels/93055209017729024/1209065403955806270/1453190691520909434

How did you test this change?

dev

@hjpalpha hjpalpha marked this pull request as ready for review December 24, 2025 04:46
Copy link
Collaborator

@ElectricalBoy ElectricalBoy left a comment

Choose a reason for hiding this comment

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

might be a good idea to exclude tournaments with this new option enabled from Module:TournamentsSummaryTable too

the stored var is either bool or nil, no readBool needed
@mbergen
Copy link
Collaborator

mbergen commented Dec 26, 2025

I dislike this feature, it's going to be abused and i don't see a good reason to add it in the first place. We have the ability to filter tickers by tier and tiertype, why isn't that sufficient?.

As far as i can tell (Not much info during LP downtime), the discord conversation starter is a olympic-style tournament, in the end being a single tournament with multiple disciplines (cf. AoElympics on AoE wiki for something similar).

@hjpalpha
Copy link
Collaborator Author

hjpalpha commented Jan 2, 2026

I dislike this feature, it's going to be abused and i don't see a good reason to add it in the first place. We have the ability to filter tickers by tier and tiertype, why isn't that sufficient?.

As far as i can tell (Not much info during LP downtime), the discord conversation starter is a olympic-style tournament, in the end being a single tournament with multiple disciplines (cf. AoElympics on AoE wiki for something similar).

¯_(ツ)_/¯
i think it is okayish to add
if we do not want it just close the PR^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants