WIP: Tabletop-Esque Exclusive Campaign Infrastructure #561

Closed

LukeZurg22 wants to merge 16 commits from campaign-whitelist into master

pull from: campaign-whitelist

merge into: Byrd-Station:master

Byrd-Station:master

Byrd-Station:ert-drafting-20260731

Byrd-Station:hitman-name-20260731

Byrd-Station:aac-improved-20260730

Byrd-Station:resomi

Byrd-Station:radio-rework-20260708

Byrd-Station:scriphud

Byrd-Station:anomalocarids

Byrd-Station:automated-upstream-branch

Conversation 0 Commits 16 Files changed 11 +653 -201

LukeZurg22 commented 2026-07-05 10:05:36 +00:00

Contributor

Copy link

PR Changes

Adds Campaign-specific player whitelist.

Reasons Behind Changes

There are people interested in a D&D-esque mini campaign, and it is about high time I add some infrastructure that has proven to be effective elsewhere.

Code & Technical details

The database model was changed to accommodate this new list. A lot of refactors were made to the Model.cs and ServerDb classes; purely syntactical.

Images; Sprites; Media

none

Checklist

• I read the Pull Request and Changelog Guidelines and Byrd Station's Contributor Ruleset am following them.
• I have added media to this PR, or it does not require an ingame showcase.
• Add temporary GM permissions. Add permissions to GM on bunker "campaign mode", remove them on disable or round restart.
• Add campaign-specific players? Separate GMs and their players lists by using GM name as key and detect if GM is present for them to join?
• Finish commands and calls for [de]assigning temporary GM status.
• Add localization.
• Run add-migration.ps1 in powershell with arbitrary name; no date.
• Test. (NON-GM NON-LISTED PLAYER FIRST. LISTED PLAYER SECOND.)

Note to self: Consider adding a timer. If a campaign session goes on too long, forcibly reopen the server and end their session. They can continue another time. Consider a timer for 24 hours to prevent instantly turning it back on? Also consider scheduling command to automate the process for close and reopen, and schedule close command to cancel it?

Consider a ForceCampaignRound command, where it starts a fresh round in Sandbox which will last for time provided?
Consider allowing non-campaigngoers to join and spectate, perhaps even ghost role. This partially solves the isolative effect.

Breaking changes

Ingame Changelog

🆑

• add: Added Campaign-Specific Whitelist

## PR Changes Adds Campaign-specific player whitelist. ## Reasons Behind Changes There are people interested in a D&D-esque mini campaign, and it is about high time I add some infrastructure that has proven to be effective elsewhere. ## Code & Technical details The database model was changed to accommodate this new list. A lot of refactors were made to the Model.cs and ServerDb classes; purely syntactical. ## Images; Sprites; Media none ## Checklist <!--Understand that not following all Contributor rule standards may get your PR closed at a maintainer’s discretion. Failure to follow the Contributor's ruleset may lead to your removal as a contributor.--> <!-- Confirm by placing an X in the brackets [X]: --> - [X] I read the [Pull Request and Changelog Guidelines](https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html) and [Byrd Station's Contributor Ruleset](https://docs.google.com/document/d/1oxCSRBc_2IlrcykljB5SPt783uJX7rMHTT7cImxlKFM/edit?tab=t.0#heading=h.vigrxrnffwji) am following them. - [X] I have added media to this PR, or it does not require an ingame showcase. - [ ] Add temporary GM permissions. Add permissions to GM on bunker "campaign mode", remove them on disable or round restart. - [ ] Add campaign-specific players? Separate GMs and their players lists by using GM name as key and detect if GM is present for them to join? - [ ] Finish commands and calls for [de]assigning temporary GM status. - [ ] Add localization. - [ ] Run add-migration.ps1 in powershell with arbitrary name; no date. - [ ] Test. (NON-GM NON-LISTED PLAYER FIRST. LISTED PLAYER SECOND.) Note to self: Consider adding a timer. If a campaign session goes on too long, forcibly reopen the server and end their session. They can continue another time. Consider a timer for 24 hours to prevent instantly turning it back on? Also consider scheduling command to automate the process for close and reopen, and schedule close command to cancel it? Consider a ForceCampaignRound command, where it starts a fresh round in Sandbox which will last for <X> time provided? Consider allowing non-campaigngoers to join and spectate, perhaps even ghost role. This partially solves the isolative effect. ## Breaking changes <!-- List any breaking changes, including namespaces, public class/method/field changes, prototype renames; and provide instructions for fixing them. This will be posted in #codebase-changes. --> **Ingame Changelog** <!-- Add a Changelog entry to make players ingame aware of new changes that could affect gameplay.--> :cl: - add: Added Campaign-Specific Whitelist

LukeZurg22 added the

size/M

documentation

bug

enhancement

labels 2026-07-05 10:05:36 +00:00

LukeZurg22 added 5 commits 2026-07-05 10:05:36 +00:00

Minor Refactors 29e11c8b11

Added CampaignList to ServerDB Model and Base Class 501956afcf

Added Campaign List Commands and CVar 956aee7cb9

Refactored Model.cs after ServerManager dff9c43cbf

Set to Campaign List Calls c56ed02ba8

LukeZurg22 self-assigned this 2026-07-05 10:05:38 +00:00

LukeZurg22 requested review from Owners 2026-07-05 10:05:44 +00:00

LukeZurg22 changed title from WIP: Tabletop-Esque Exclusive Campaign to WIP: Tabletop-Esque Exclusive Campaign Infrastructure 2026-07-05 10:08:45 +00:00

LukeZurg22 added 1 commit 2026-07-05 10:09:36 +00:00

Cleanup CampaignListCommands.cs 5959f6cba0

LukeZurg22 added 4 commits 2026-07-05 10:45:24 +00:00

Teensy Refactor to ConnectionManager.cs 9aac5554a7

Syntactical Refactors af154364ce

Added campaign-exclusive connection denial. 51d46c8686

Cleanup wording in CCVars.Game.cs 2b76c83567

LukeZurg22 added 4 commits 2026-07-05 10:58:09 +00:00

Added Admin bypass to Campaign Bunker. 21fa1f9f9c

Disabled inconsistent naming warning in CDModel.cs fa7aae8a72

Updated ConnectionManager.Whitelist.cs 7e49f151dc

Added CampaignHost player profile check. ... 0834aa9fd7

Note: VERY important! Used for permissions and whatnot later.

LukeZurg22 added 1 commit 2026-07-05 11:03:19 +00:00

Adjust Model.cs so that campaign host isn't character-specific. bbbdcf3900

LukeZurg22 added 1 commit 2026-07-05 11:31:46 +00:00

Grammar adjustment in ServerDbManager.cs a73a5fcfe1

LukeZurg22 added the

DON'T MERGE

label 2026-07-17 23:41:33 +00:00

LukeZurg22 closed this pull request 2026-07-24 04:32:14 +00:00

All checks were successful

Hide all checks

Test Packaging / Test Packaging (pull_request) Has been skipped

Details

CRLF Check / CRLF Check (pull_request) Has been skipped

Details

RGA schema validator / YAML RGA schema validator (pull_request) Has been skipped

Details

Labels: Size / size-label (pull_request_target) Successful in 3s

Details

Labels: PR / labeler (pull_request_target) Successful in 3s

Details

Map file schema validator / YAML map schema validator (pull_request) Has been skipped

Details

YAML Linter / YAML Linter (pull_request) Has been skipped

Details

RSI Validator / Validate RSIs (pull_request) Successful in 58s

Details

PR Changelogs / changelog (pull_request_target) Has been skipped

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Byrd-Station/Owners

Labels

Clear labels   

bug

This is in regards to a bug.

  

Changes: Audio

  

Changes: Map

  

Changes: No C#

  

Changes: Sprites

  

Changes: UI

  

documentation

Improvements or additions to documentation

  

DON'T MERGE

This is under review, fixes, or needs significant changes.

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

holding to vote

This is currently being voted for by the Byrds.

  

invalid

This doesn't seem right

  

question

Further information is requested

  

size/L

  

size/M

  

size/S

  

size/XL

  

size/XS

  

S: Merge Conflict

  

S: Untriaged

  

wontfix

This will not be worked on

No labels

bug

Changes: Audio

Changes: Map

Changes: No C#

Changes: Sprites

Changes: UI

documentation

DON'T MERGE

duplicate

enhancement

good first issue

help wanted

holding to vote

invalid

question

size/L

size/M

size/S

size/XL

size/XS

S: Merge Conflict

S: Untriaged

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

LukeZurg22

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Byrd-Station/Byrd!561

No description provided.