Skip to content

feat(match2): add dev environment support for br - #7996

Open
ElectricalBoy wants to merge 4 commits into
mainfrom
match2-br-dev-support
Open

feat(match2): add dev environment support for br#7996
ElectricalBoy wants to merge 4 commits into
mainfrom
match2-br-dev-support

Conversation

@ElectricalBoy

Copy link
Copy Markdown
Collaborator

Summary

Match2 BR/FFA did not support directly applying dev environment arguments to game tabs as they were loaded via javascript due to performance reasons.
This PR exports the dev flag to the generated matchgroup display html code, allowing it to be picked up by javascript.

How did you test this change?

dev in pubg + dev proxy

@ElectricalBoy
ElectricalBoy requested review from a team and liquidely August 21, 2026 01:51
@Rathoz

Rathoz commented Aug 24, 2026

Copy link
Copy Markdown
Member

We have this for hte main page match ticker too. Is it feasible to implement a common usage that can be easily exnended in the future to other components if the need arises?

@mbergen

mbergen commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

We have this for hte main page match ticker too. Is it feasible to implement a common usage that can be easily exnended in the future to other components if the need arises?

Probably not, for the matchticker (or rather, filterbutton template expansion one can supply anything that fits into {{<here>|paramFromFilter}}, so the the lua code just appends the dev flag as needed, JS does not even know.
Due to the setup of BR JS being quite a bit more complicated, i don't see a way to use the same thing here.

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