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

fix: vendor add buy and add buyp commands #11

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

Conversation

jensbodal
Copy link

vendor add buy and add buyp commands should be fixed.

Added a new vendor buymax command

  • Buys the maximum amount of an item (by exact name) based on your current
  • pyreal count.
  • Need to specify cost of item

e.g.

/mt vendor addmax Trade Note (250,000)|287500

@jensbodal jensbodal changed the title fix: fix vendor add buy and add buyp commands fix: vendor add buy and add buyp commands Jan 20, 2019
vendor `add buy` and `add buyp` commands should be fixed.

Added a new vendor `buymax` command
  * Buys the maximum amount of an item (by exact name) based on your current
  * pyreal count.
  * Need to specify cost of item

e.g.

```
/mt vendor addmax Trade Note (250,000)|287500
```
@Mag-nus
Copy link
Owner

Mag-nus commented Feb 18, 2019

Can you elaborate on what the issue with "add buy" and "add buyp" were?

Also, the buymax command is very nice.

@jensbodal
Copy link
Author

To put it simply, the commands did not work for me. I'm not sure if this has something to do with the emulator (gdleac).

From what I remember from retail I'm not sure how this would work without subscribing to the approach vendor filter so that you can obtain the WOs from their inventory. The vendor WOs aren't available just because the vendor is open, I believe they're only available as part of the event which triggers upon opening the vendor.

I'm sure these commands worked before as they wouldn't be there otherwise, but I wasn't able to get them to work until I did it his way.

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.

None yet

2 participants