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

workorders - expired workorder round 2 #366

Closed
OdnaZvezda opened this issue Apr 26, 2016 · 2 comments
Closed

workorders - expired workorder round 2 #366

OdnaZvezda opened this issue Apr 26, 2016 · 2 comments

Comments

@OdnaZvezda
Copy link
Contributor

Found the root of the problem. Crafted items were too low quality for the workorder requirement and were dropped rather than bundled with the workorder.

First, workorders only creates as many items as the workorder calls for, then tries to give the logbook to the society craftmaster. http://pastebin.com/8uYSwQT6

#362 was happening as a result of the above.

Better knitting needles should fix this for the character in question.

@rcuhljr
Copy link
Collaborator

rcuhljr commented Apr 26, 2016

Yeah it's kind of a known issue that if you're tanking items it won't work. I haven't gone through and done the leg work to deal with failures although I think I I could add in some logic pretty cleanly.

@OdnaZvezda
Copy link
Contributor Author

I don't know that dealing with failures is necessary. If you don't have the skill to consistently craft the items at the quality needed, it's reasonable to assume that you'd run out of time while crafting replacements. Workorders could possibly echo a message to the effect that the crafted items were too low quality for the logbook and return control to crossing-training

@rpherbig rpherbig added this to the v5 milestone May 2, 2016
@rpherbig rpherbig modified the milestones: v6, v5 Jun 29, 2016
@rpherbig rpherbig modified the milestones: v7, v8, v9 Sep 16, 2016
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

3 participants