Skip to content

feat(infobox): Add Clarioncorp links#7053

Open
Hesketh2 wants to merge 4 commits intomainfrom
H2-ClarionCorp
Open

feat(infobox): Add Clarioncorp links#7053
Hesketh2 wants to merge 4 commits intomainfrom
H2-ClarionCorp

Conversation

@Hesketh2
Copy link
Collaborator

@Hesketh2 Hesketh2 commented Feb 5, 2026

Summary

Links for new website Clarioncorp
image

This PR includes an attempt to redirect whenever |corestrike= is used on a page, this should return the link and icon for |clarioncorp= as the player name input will still work.

How did you test this change?

Currently the alias redirection doesnt work, idk way but the exact clarioncorp input does work (with placeholder icons)
https://liquipedia.net/omegastrikers/Slof

Sidenote

Is it a better option to just not do aliases and request for bot param edits once this is merged? (185 pages under Corestrike)

@Hesketh2 Hesketh2 self-assigned this Feb 5, 2026
@Hesketh2 Hesketh2 requested review from a team February 5, 2026 15:54
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.

should the corestrike icon css be kicked too?

is this discussed with cs admins (mnmzz, imarbot, ...)?

@Hesketh2
Copy link
Collaborator Author

Hesketh2 commented Feb 5, 2026

should the corestrike icon css be kicked too?

is this discussed with cs admins (mnmzz, imarbot, ...)?

Can Kick if it doesnt break other things
not sure if I will need to check with CS admins, Corestrike is not CS-related site

@hjpalpha
Copy link
Collaborator

hjpalpha commented Feb 5, 2026

should the corestrike icon css be kicked too?
is this discussed with cs admins (mnmzz, imarbot, ...)?

Can Kick if it doesnt break other things not sure if I will need to check with CS admins, Corestrike is not CS-related site

ah my bad assumed "strike" —> cs

local ALIASES = {
['ask-fm'] = {'afk.fm', 'askfm'},
douyu = {'douyutv'},
corestrike = {'clarioncorp'},
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wrong way around, it's target = {aliases}

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.

3 participants