Skip to content

Whitelist alternatives in the config#2001

Merged
drtheodor merged 1 commit into
amblelabs:mainfrom
Peter1169:feat/whitelist-config
Mar 24, 2026
Merged

Whitelist alternatives in the config#2001
drtheodor merged 1 commit into
amblelabs:mainfrom
Peter1169:feat/whitelist-config

Conversation

@Peter1169
Copy link
Copy Markdown
Contributor

About the PR

Adds alternative whitelists for the four existing dimension blacklists:

  • Environment Projector
  • TARDIS travel
  • Rift spawn
  • Rift drop

If a whitelist has entries, it overrides its matching blacklist.

Why / Balance

This gives modpack/server owners a cleaner allow-only configuration for dimensions. Large modpacks no longer have to manually enter 47 dimension entries in the blacklist and can just use the whitelist instead.
Balance is unchanged unless these new config entries are used, since all whitelists default to empty.

Technical details

Added four new server config lists and their config screen translations.
Updated world cache generation so whitelist entries take priority over blacklists, while ignoring empty/invalid entries safely.
Also added a few guards so empty resolved lists do not cause projector, travel control, or rift-related runtime issues.

Media

N/A

Requirements

Breaking changes

N/A

Changelog
🆑

  • tweak: Alternative dimension whitelists in the config for the Environment Projector, TARDIS travel, rift spawn, and rift drop; whitelists override their matching blacklists when populated.

@Peter1169 Peter1169 requested review from a team as code owners March 23, 2026 17:49
@github-actions github-actions Bot added S: Needs Review Status: Requires additional reviews before being fully accepted. size/M Denotes a PR that changes 100-999 lines. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. A: Tardis Components Area: Tardis components & manager. A: Console Area: Tardis console. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/M Denotes a PR that changes 100-999 lines. S: Needs Review Status: Requires additional reviews before being fully accepted. labels Mar 23, 2026
Copy link
Copy Markdown
Member

@drtheodor drtheodor left a comment

Choose a reason for hiding this comment

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

LGTM, tested?

@Peter1169
Copy link
Copy Markdown
Contributor Author

yes

@duzos duzos self-requested a review March 23, 2026 20:38
@github-actions github-actions Bot added the S: Needs Review Status: Requires additional reviews before being fully accepted. label Mar 23, 2026
Copy link
Copy Markdown
Member

@duzos duzos left a comment

Choose a reason for hiding this comment

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

LGTM

@drtheodor drtheodor merged commit 4308e0e into amblelabs:main Mar 24, 2026
7 of 10 checks passed
@Peter1169 Peter1169 deleted the feat/whitelist-config branch March 25, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: Console Area: Tardis console. A: Tardis Components Area: Tardis components & manager. S: Needs Review Status: Requires additional reviews before being fully accepted.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants