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

Eddy swap script #6741

Closed
wants to merge 4 commits into from
Closed

Eddy swap script #6741

wants to merge 4 commits into from

Conversation

Nyt3
Copy link
Contributor

@Nyt3 Nyt3 commented Dec 1, 2023

Adding a newly created eddy-swap script. This script is intended get items from/put items into the portal. The script supports using containers in the portal for organization as an optional setting.

There are a number of checks in the script to prevent loss of items. Containers are validated before actions are taken, items are tapped prior to being move in either direction, the the existence of the portal itself is also checked.

The base.yaml updates include placeholder examples to demonstrate the configuration settings. Placeholders were used in the event someone correctly ran the script accidently and moved items without warning.

@mwest152
Copy link
Contributor

Are you wanting this to transfer items between bags, like the transfer-item script? Or specifically for storing tools? I have a working version of the clerk-tools script that will use your eddy instead of the clerk. I just don't really know how to do pull requests and stuff.

@Nyt3
Copy link
Contributor Author

Nyt3 commented Dec 30, 2023

The original intent for this script was two-fold:

  1. Allow a list of items to be stored/removed from the portal.
  2. Allow a list of items to be stored/removed from a container within the portal (for organizational purposes)

As a side point, I wanted the list of items to be treated in a similar fashion as the gearsets (e.g., store combat gear, store enchanting tools, etc.) but with some added flexibility of having containers in the portal (e.g., all crafting tools can be a dergantine backpack in the portal, combat gear in a haversack, formal RP wear in a satchel, etc.)

I also based this on the clerk-tools script, but I'm happy to pull this back. I did receive some feedback so it definitely needs to be retooled before it's merged. If you have an already working solution, I think that would be great!

@mwest152
Copy link
Contributor

the script transfer-items does some of that. Check out this custom version of clerk tools that will use an eddy instead of the actual clerk as well.
clerk-tools - Copy.txt

@MahtraDR
Copy link
Collaborator

MahtraDR commented Apr 8, 2024

What's the plan here? Adjust and chug on, or nix this in favour of transfer-items?

@Nyt3
Copy link
Contributor Author

Nyt3 commented Jun 7, 2024

Closing this request until I do some clean-up and updates.

@Nyt3 Nyt3 closed this Jun 7, 2024
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.

3 participants