Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin This PR affects administrators Fix This fixes an issue. Please link issues in fix PRs
Projects
None yet
3 participants