Skip to content
Snippets Groups Projects

Add warning and reset flow for non-default masterserver address

Merged AJ Martinez requested to merge ms-address-warning into master
All threads resolved!

For the sake of every brave warrior in #tech-support, watching people blindly flail at their keyboards and fail to port forward.

srb2kart_MGKU1r4RDG

This adds a prompt to reset masterserver whenever a player attempts to host a netgame or load the in-game Server Browser with a non-default masterserver. If the player mashes through it, the default behavior is to reset, because nobody reads anything and I honestly cannot blame them at all.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Compiled it, it works and the intent is clear.

    But given this is also for futureproofing, maybe an extra cvar to disable the nagging would be good too.

  • AJ Martinez added 1 commit

    added 1 commit

    • 8548d441 - Futureproofing mserv address conveyance

    Compare with previous version

  • AJ Martinez added 1 commit

    added 1 commit

    • 1441fda8 - Clearly mark MS reminder + formatting touchup

    Compare with previous version

  • Author Maintainer

    Added masterserver_nagattempts, default 5. Reduces by 1 when you dismiss the "wrong address" nag, once per boot. At 0, nags are disabled. Savvy players can disable the nags right away, but a future player on a community Master Server would eventually stop being harassed even without knowing about this.

    To cover for someone somehow dismissing this without reading it 5 times (????), and for better general conveyance, the Server Browser address is now displayed when setting up a netgame with advertise On, or when viewing the in-game Server Browser. It's transparent and uses your highlight color when default, but uses the warning color and full opacity if it's changed.

    srb2kart_3MsG93JKys

    srb2kart_7j5hlhWaaR

    srb2kart_VtjywWQTHi

    srb2kart_NCjrjTFu0Z

  • JugadorXEI
  • AJ Martinez added 1 commit

    added 1 commit

    Compare with previous version

  • AJ Martinez resolved all threads

    resolved all threads

  • merged

  • toaster mentioned in commit 34fd13be

    mentioned in commit 34fd13be

  • Please register or sign in to reply
    Loading