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

seedwatch can't handle pomegranate seeds #4504

Closed
myk002 opened this issue Apr 23, 2024 · 1 comment · Fixed by #4650
Closed

seedwatch can't handle pomegranate seeds #4504

myk002 opened this issue Apr 23, 2024 · 1 comment · Fixed by #4650

Comments

@myk002
Copy link
Member

myk002 commented Apr 23, 2024

as per https://discord.com/channels/793331351645323264/807444467140788254/1232210454323925003

@ab9rf
Copy link
Member

ab9rf commented May 30, 2024

pomegranates are trees, and so pomegranate seeds are tree seeds; seedwatch doesn't do anything for tree seeds because tree seeds are not plantable and so there is no purpose in retaining a stock of them

the reason it reports that "POMEGRANATE has not been found as a material" is because it's looking for a plantable seed and pomegranate seeds are not plantable. there might also be something weird about pomegranates in the raws here, need to research this further

so this is WAI except perhaps that seedwatch should simply not mention tree seeds at all since it has nothing useful to say or do about them anyway, which i thought we had done but apparently while we fixed it in one place we missed it in the other

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants