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

Debug craft #1

Open
Muirrum opened this issue Oct 26, 2023 · 0 comments
Open

Debug craft #1

Muirrum opened this issue Oct 26, 2023 · 0 comments
Milestone

Comments

@Muirrum
Copy link
Member

Muirrum commented Oct 26, 2023

Currently returns

[RecipeIndex]: '0' | Needs to implement 'required'%

When trying to craft a plant

@Muirrum Muirrum added this to the 1.0 milestone Oct 26, 2023
Muirrum added a commit that referenced this issue Mar 14, 2024
Issues were with not correctly casting the final reference to the recipe
(needed to be cast to map[string]interface{} but wasn't, so was instead
interface{}, a non-indexable type).

There's currently an issue with TakeItem where it's not fetching the
inventory correctly from the database. That needs further investigation.
I suspect it's something to do with db.Get not correctly interpreting
the JSON data.

Also, the issue with `recipe_index` not passing the Required check was
due to the validator interpreting indicies of "1" as invalid. More
investigation required.

TODO: Debug Required check on recipe_index
TODO: Investigate TakeItem issues as noted above
Issue: #1
Signed-Off-By: Cara Salter <[email protected]>
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

No branches or pull requests

1 participant