Skip to content

Add RocketHost lobby browser#101

Merged
VirxEC merged 4 commits into
masterfrom
rhost-update
May 25, 2026
Merged

Add RocketHost lobby browser#101
VirxEC merged 4 commits into
masterfrom
rhost-update

Conversation

@VirxEC
Copy link
Copy Markdown
Contributor

@VirxEC VirxEC commented May 25, 2026

  • As a temporary helper with the growing list of RocketHost bots, the number of columns of bots has been upped to 3 and resizes itself based on screen width:
    image
    image
    image

  • When no RocketHost servers are listed, the join row is replaced with a down for maintenance message:
    image
    image

  • Added RocketHost lobby browser that allows the user to join existing lobbies
    image
    image

@NicEastvillage
Copy link
Copy Markdown
Contributor

Looking good so far.

How about a "refresh" button in the lobby browser? I know you can close it and open it again, but a refresh button makes it clear that it doesn't happen automatically.

Do we have more ways to distinguish the servers? It is possible that two of them are essentially identical in the browser.

@VirxEC
Copy link
Copy Markdown
Contributor Author

VirxEC commented May 25, 2026

The only way for us to distinguish the servers is their port number, which I thought might confuse normal users - what is this random number doing after the lobby name? Although we indeed might just need to add it, due to it being the only unique identifier...

@NicEastvillage
Copy link
Copy Markdown
Contributor

As far as I can tell, the host is not shown the port of their server, so the host can say which port to use anyway. Could we list the bots/humans in the match?

@VirxEC
Copy link
Copy Markdown
Contributor Author

VirxEC commented May 25, 2026

Example API return:

{
  "lobbies": [
    {
      "id": "ea033c17-31db-4693-8246-56af0d19d4a4",
      "name": "RH-US-C-RLBot:6438",
      "map": "ARC_Darc_P",
      "description": "",
      "playerCount": 1,
      "players": [],
      "hasPassword": false,
      "ipAddress": "154.7.120.61",
      "port": 6438,
      "secondsSinceUpdate": 131
    },
    {
      "id": "28e773c6-7a28-4092-812a-88ff54d9d6c8",
      "name": "RH-US-C-RLBot:6447",
      "map": "Stadium_P",
      "description": "",
      "playerCount": 1,
      "players": [],
      "hasPassword": false,
      "ipAddress": "154.7.120.61",
      "port": 6447,
      "secondsSinceUpdate": 11
    },
    {
      "id": "328702ef-24e4-4457-8f91-d3db38942e36",
      "name": "RH-US-C-RLBot:6444",
      "map": "ARC_Darc_P",
      "description": "",
      "playerCount": 1,
      "players": [],
      "hasPassword": false,
      "ipAddress": "154.7.120.61",
      "port": 6444,
      "secondsSinceUpdate": 9
    },
    {
      "id": "3d3822b0-1876-4ce7-b560-a66d1960c64e",
      "name": "RH-US-C-RLBot:6441",
      "map": "Stadium_P",
      "description": "",
      "playerCount": 1,
      "players": [],
      "hasPassword": false,
      "ipAddress": "154.7.120.61",
      "port": 6441,
      "secondsSinceUpdate": 98
    }
  ]
}

players is ALWAYS blank, description is blank, and has password is always false lol

@VirxEC
Copy link
Copy Markdown
Contributor Author

VirxEC commented May 25, 2026

tbh this isn't that bad
image

but I initially avoided added a refresh button because it adds more complexity to the code (I was being lazy)

@NicEastvillage
Copy link
Copy Markdown
Contributor

Yeah, let's add the port. It's fine.

@VirxEC VirxEC merged commit 74bc5d8 into master May 25, 2026
@VirxEC VirxEC deleted the rhost-update branch May 25, 2026 16:03
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.

2 participants