Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

[Admin] VV will no longer give you nuisance alerts about getting anything from the same Z-level if you don't have +FUN#22809

Merged
JamieD1 merged 1 commit intoyogstation13:masterfrom
Oblisk234:vvget
Nov 21, 2024
Merged

[Admin] VV will no longer give you nuisance alerts about getting anything from the same Z-level if you don't have +FUN#22809
JamieD1 merged 1 commit intoyogstation13:masterfrom
Oblisk234:vvget

Conversation

@Oblisk234
Copy link
Member

@Oblisk234 Oblisk234 commented Nov 16, 2024

Fixes #22795

Document the changes in your pull request

The ifs checking whether it was an item on CC and whether you have +FUN was the wrong way round.

Testing

Still checks if you're getting something from CC and stops you if you don't have +FUN
image
Getting things from CC with +FUN still works:
image

Changelog

🆑
bugfix: VV getting anything on the same Z-level without +FUN will no longer give you nuisance alerts about not having +FUN.
/:cl:

@Yogbot-13 Yogbot-13 added the Fix This fixes an issue. Please link issues in fix PRs label Nov 16, 2024
@github-actions github-actions bot added the Admin This PR affects administrators label Nov 16, 2024
@JamieD1 JamieD1 merged commit c66312b into yogstation13:master Nov 21, 2024
Yogbot-13 added a commit that referenced this pull request Nov 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Admin This PR affects administrators Fix This fixes an issue. Please link issues in fix PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VV getting a mob without having +FUN gives you a nuisance message about not having +FUN while still allowing you to get the mob

4 participants