Skip to content

Show minimum version in update dialog#4888

Open
omar-o22 wants to merge 3 commits intomultitheftauto:masterfrom
omar-o22:show_minimum_version
Open

Show minimum version in update dialog#4888
omar-o22 wants to merge 3 commits intomultitheftauto:masterfrom
omar-o22:show_minimum_version

Conversation

@omar-o22
Copy link
Copy Markdown
Contributor

@omar-o22 omar-o22 commented May 4, 2026

Summary

Displays the minimum required client version in the update dialog when a user tries joins a server with a higher minclientversion.

Motivation

Sometimes users update failures, displaying the required client version helps anyone assisting them identify which version is needed, so it can be installed manually.

Test plan

  1. Connect to a server with a higher minclientversion than the client.

Checklist

  • Your code should follow the coding guidelines.
  • Smaller pull requests are easier to review. If your pull request is beefy, your pull request should be reviewable commit-by-commit.

Comment thread Client/core/CVersionUpdater.cpp Outdated
@omar-o22 omar-o22 requested a review from qaisjp May 4, 2026 14:40
qaisjp
qaisjp previously approved these changes May 10, 2026
Copy link
Copy Markdown
Member

@qaisjp qaisjp left a comment

Choose a reason for hiding this comment

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

lgtm but tests aren't passing

@omar-o22
Copy link
Copy Markdown
Contributor Author

lgtm but tests aren't passing

fixed

Copy link
Copy Markdown

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.

Pull request overview

Updates the client-side mandatory update prompt shown when a server rejects a connection due to minclientversion, so the dialog includes the required version string provided by the server.

Changes:

  • Updated the “MANDATORY UPDATE” dialog message to include the server-provided required client version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

GetQuestionBox().Reset();
GetQuestionBox().SetTitle(_("MANDATORY UPDATE"));
GetQuestionBox().SetMessage(_("To join this server, you must update MTA.\n\n Do you want to update now ?"));
GetQuestionBox().SetMessage(SString(_("To join this server, you must update MTA to version %s.\n Do you want to update now?"), *m_strServerSaysData));
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should I do that? @qaisjp

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