Skip to content

ISR - Add setting to disable menu#11

Merged
BrettMayson merged 7 commits into
BrettMayson:mainfrom
SpicyBagpipes:patch-1
Jun 28, 2026
Merged

ISR - Add setting to disable menu#11
BrettMayson merged 7 commits into
BrettMayson:mainfrom
SpicyBagpipes:patch-1

Conversation

@SpicyBagpipes

Copy link
Copy Markdown
Contributor

No description provided.

@SpicyBagpipes
SpicyBagpipes marked this pull request as draft May 4, 2026 14:15
@SpicyBagpipes

Copy link
Copy Markdown
Contributor Author

nvm im stupid i forgot signal is a different addon

@SpicyBagpipes
SpicyBagpipes marked this pull request as ready for review May 4, 2026 14:17
@BrettMayson

Copy link
Copy Markdown
Owner

This still doesn't get to the root issue you're having, which is ISR on turrets.

This will also disable the ability of UAV operators to use the menus, and change frequency and power settings

@SpicyBagpipes

SpicyBagpipes commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

This still doesn't get to the root issue you're having, which is ISR on turrets.

This will also disable the ability of UAV operators to use the menus, and change frequency and power settings

I want the ability to disable it for those anyway, auto adjust is fine for my purposes

I can just change it into more settings if you'd rather, but am cautious of settings spam

@SpicyBagpipes

Copy link
Copy Markdown
Contributor Author

ISR on turrets I'm not sure is fixable without doing some weird checks - if it's intended to only be for UAVs, I can just add a UAV check, but if you want it for more than just that then we're in the realm of stuff like datalink checks or something, which is pretty opinionated

@BrettMayson

Copy link
Copy Markdown
Owner

I think this could be a drop down, of enabled, disabled, uav only

@SpicyBagpipes

Copy link
Copy Markdown
Contributor Author

good idea, will do in the next couple of days, arrived on vacation at start of week

@SpicyBagpipes

Copy link
Copy Markdown
Contributor Author

To me this seems like the least convoluted approach, though I'm not 100% on the readability

@BrettMayson
BrettMayson merged commit 3b3cf05 into BrettMayson:main Jun 28, 2026
@SpicyBagpipes
SpicyBagpipes deleted the patch-1 branch June 30, 2026 22:40
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