Skip to content

Disable fusion admin room + new helper - #11

Merged
StrawWagen merged 3 commits into
mainfrom
gm_darkfusion
Feb 15, 2026
Merged

Disable fusion admin room + new helper#11
StrawWagen merged 3 commits into
mainfrom
gm_darkfusion

Conversation

@StrawWagen

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request refactors admin button removal logic into a reusable utility function and applies it to disable admin rooms in two maps: gm_vacant_industry_revamped (existing) and gm_darkfusion (new).

Changes:

  • Added Utils.RemoveButtonsByID() helper function to centralize button removal logic with validation
  • Refactored gm_vacant_industry_revamped map script to use the new utility function
  • Added new map script for gm_darkfusion to disable its admin room buttons

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
lua/map_scripts/utils.lua Added new RemoveButtonsByID() utility function with button class validation and error handling
lua/map_scripts/maps/gm_vacant_industry_revamped.lua Refactored to use new utility function, removing duplicate logic; fixed variable naming from entIds to entIDs
lua/map_scripts/maps/gm_darkfusion.lua New map script to remove admin buttons using the utility function

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

Comment thread lua/map_scripts/maps/gm_darkfusion.lua Outdated
Comment thread lua/map_scripts/maps/gm_darkfusion.lua Outdated
@StrawWagen

Copy link
Copy Markdown
Contributor Author

Tested, works

@StrawWagen

Copy link
Copy Markdown
Contributor Author

yolo

@StrawWagen
StrawWagen merged commit bdabc26 into main Feb 15, 2026
1 check passed
@StrawWagen
StrawWagen deleted the gm_darkfusion branch February 15, 2026 07:29
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