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

redux: sell all /class/ functionality #146

Open
3 tasks
jpcsupplies opened this issue Jul 12, 2016 · 0 comments
Open
3 tasks

redux: sell all /class/ functionality #146

jpcsupplies opened this issue Jul 12, 2016 · 0 comments

Comments

@jpcsupplies
Copy link
Owner

Just moved this here from the shortened lazy commands issue i added already.

Possible areas of improvement outside my own skill level -
Midspace whats your opinion on these - worth doing? Probably a bit too fiddly for my skill level at the moment -

  • /sell "ALL" tag "keywords". Specify an item class. ie ingot ore component tool ammo eg "/sell all ingots" - or when using "ALL" have it sell all matching items instead of "did you mean...." messages eg "/sell all comp" would sell construction components and computers etc.
  • item class probably should have some restriction, eg is player an admin, OR does player own the market OR player owns all cargo crates containing items of that class. Otherwise new players may jump in a seat on a faction and sell its entire inventory for grief and profit..
  • for what might be rather large sales, some sort of are you sure yes / no prompt? Perhaps using the mission window. press X to cancel otherwise ok to sell. This could also display a manifest of all the matching components in the window. (actually this sort of yes/no might be something I could knock up as a bool sub.. that sounds like fun. if i can recall the trap the X uses dang where were my notes here...)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant