Skip to content

Add custom disconnection dialog when version mismatch is a likely cause#908

Merged
notfood merged 1 commit intorwmt:devfrom
mibac138:bad-packet-id
May 10, 2026
Merged

Add custom disconnection dialog when version mismatch is a likely cause#908
notfood merged 1 commit intorwmt:devfrom
mibac138:bad-packet-id

Conversation

@mibac138
Copy link
Copy Markdown
Member

@mibac138 mibac138 commented May 8, 2026

A rather niche issue, but probably won't hurt to have it. Not sure if the DebugActions are worth it, probably overkill?

Without Steam:
image

With Steam:
image

@mibac138 mibac138 added enhancement New feature or request. 1.6 Fixes or bugs relating to 1.6 (Not Odyssey). labels May 8, 2026
@notfood
Copy link
Copy Markdown
Member

notfood commented May 9, 2026

People who encounter this are most likely not playing with Steam.

The debug actions are overkill, when are we going to use that?

@mibac138
Copy link
Copy Markdown
Member Author

mibac138 commented May 9, 2026

The debug actions are overkill, when are we going to use that?

Yeah, thought as much. I just used them to create this and it was useful in the short span of time I was creating this :)

People who encounter this are most likely not playing with Steam.

I do agree, but sometimes Steam is wonky with mod updates and refuses to acknowledge a mod update is available. I had this issue once and calling SteamUGC.DownloadItem made Steam actually detect (and download) the update. Though that's probably not a particularly common case, so I'm fine with removing it if you're against it

@notfood
Copy link
Copy Markdown
Member

notfood commented May 9, 2026

That note should be sufficient. The multiplayer mod doesn’t update often enough and protocol doesn't break with every update for this to be an issue, so a force update button shouldn't be necessary.

@mibac138
Copy link
Copy Markdown
Member Author

mibac138 commented May 9, 2026

Yup, removed it :)

@notfood
Copy link
Copy Markdown
Member

notfood commented May 9, 2026

Don't forget to PR Multiplayer-Locale

@notfood notfood merged commit b8bf25c into rwmt:dev May 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.6 Fixes or bugs relating to 1.6 (Not Odyssey). enhancement New feature or request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants