Skip to content

fix: Switch recent matches to player instead team#7350

Open
SzakaliOrs wants to merge 1 commit intomainfrom
r6_infobox_update_centaur
Open

fix: Switch recent matches to player instead team#7350
SzakaliOrs wants to merge 1 commit intomainfrom
r6_infobox_update_centaur

Conversation

@SzakaliOrs
Copy link
Copy Markdown
Collaborator

Summary

Changing to show the recent matches of the player not the team

How did you test this change?

Dev module:https://liquipedia.net/rainbowsix/Module:Infobox/Person/Player/Custom/dev/Okidokie98

@SzakaliOrs SzakaliOrs requested review from a team as code owners April 4, 2026 19:59
Copilot AI review requested due to automatic review settings April 4, 2026 19:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot AI stuff due to "on by default" settings on my side, could delete the rest only see an option for editing but not deleting for this one(centaur)

@Liquipedia Liquipedia deleted a comment from Copilot AI Apr 4, 2026
@Liquipedia Liquipedia deleted a comment from Copilot AI Apr 4, 2026
@SzakaliOrs SzakaliOrs changed the title Switch recent matches to player instead team fix: Switch recent matches to player instead team Apr 4, 2026
@Rathoz
Copy link
Copy Markdown
Collaborator

Rathoz commented Apr 4, 2026

Recent or upcoming?

@SzakaliOrs
Copy link
Copy Markdown
Collaborator Author

SzakaliOrs commented Apr 4, 2026

Recent matches. Currently we show recent matches and upcoming tournaments, this should(if i didn't mess anything up) only affect the recent matches part and not the upcoming tournaments part.

For example Mag joined Enterprise after SI but SI matches show up as "Recent Matches" on his page because it takes them based on the team. It would fix so it only shows the matches he played as a player.
Before:
image
After:
image

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