Skip to content

Remove auto join spectator team#267

Merged
B3none merged 3 commits intoB3none:masterfrom
Ravid-A:master
Apr 12, 2026
Merged

Remove auto join spectator team#267
B3none merged 3 commits intoB3none:masterfrom
Ravid-A:master

Conversation

@Ravid-A
Copy link
Copy Markdown
Contributor

@Ravid-A Ravid-A commented Mar 7, 2026

When you connect to the game, it immediately switches you to spectator mode, and you sit under the map until you call up the team selection window yourself.

@st1ng2
Copy link
Copy Markdown
Contributor

st1ng2 commented Mar 8, 2026

At this stage, I think it's better to leave everything as it is. Not everyone wants to join right away, perhaps because they don't want to kick anyone out because of privilege. Or it might be better to leave it as configurable parameter.

@Ravid-A
Copy link
Copy Markdown
Contributor Author

Ravid-A commented Mar 8, 2026

At this stage, I think it's better to leave everything as it is. Not everyone wants to join right away, perhaps because they don't want to kick anyone out because of privilege. Or it might be better to leave it as configurable parameter.

I understand what you're saying but the problem created from this is that the player will need to open the teammenu manually which is uncomfortable

@wiruwiru
Copy link
Copy Markdown
Contributor

wiruwiru commented Mar 8, 2026

I think you could create a new configuration option to enable/disable the logic instead of simply removing it from the code.

@Ravid-A
Copy link
Copy Markdown
Contributor Author

Ravid-A commented Mar 8, 2026

I think you could create a new configuration option to enable/disable the logic instead of simply removing it from the code.

I think it's an option for everyone's comfort but not the full solution, my solution is not the right solution

@B3none
Copy link
Copy Markdown
Owner

B3none commented Mar 8, 2026

I think you could create a new configuration option to enable/disable the logic instead of simply removing it from the code.

This is the ideal solution

@Ravid-A
Copy link
Copy Markdown
Contributor Author

Ravid-A commented Apr 12, 2026

I think you could create a new configuration option to enable/disable the logic instead of simply removing it from the code.

i just added the config option for this feature

@B3none B3none merged commit c85ca95 into B3none:master Apr 12, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants