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

[script][burgle] Added portable trashcan functionality #6889

Merged
merged 6 commits into from
Sep 20, 2024

Conversation

hat071af
Copy link
Contributor

@hat071af hat071af commented Aug 4, 2024

Added the ability to use a portable trashcan to dispose of items. To use use loot: trashcan within burgle_settings:

burgle_settings:
  loot: trashcan  # drop | keep | bin | pawn | trashcan   default: keep. trashcan is for using a portable trashcan

Also requires worn_trashcan: and worn_trashcan_verb: to also be set outside of your burgle_settings. Example settings:

worn_trashcan: silversteel bucket
worn_trashcan_verb: tap

Added the ability to use a portable trashcan to dispose of items. To use use `loot: trashcan` within `burgle_settings:`
```
burgle_settings:
  loot: trashcan  # drop | keep | bin | pawn | trashcan   default: keep, trashcan is for using a portable trashcan
```
Requires `worn_trashcan:` and `worn_trashcan_verb:` to also be set. Example settings:
```
worn_trashcan: silversteel bucket
worn_trashcan_verb: tap
```
@hat071af
Copy link
Contributor Author

hat071af commented Aug 4, 2024

Don't merge this one quite yet, please. Adjusting an interaction with whitelist and the new trashcan option.

Fixed trashcan setting bypassing whitelist items
@hat071af
Copy link
Contributor Author

hat071af commented Aug 5, 2024

Trashcan setting no longer bypasses whitelisted items. Feel free to merge upon review.

@MahtraDR MahtraDR merged commit eba58e1 into elanthia-online:master Sep 20, 2024
2 checks passed
@hat071af hat071af deleted the burgle branch September 30, 2024 07:22
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