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

Move pine resin to the chemical category #59996

Merged
merged 1 commit into from
Aug 7, 2022

Conversation

pjf
Copy link
Contributor

@pjf pjf commented Aug 7, 2022

Summary

Bugfixes "Pine resin is now in the chemical category"

Purpose of change

Pine resin previously didn't have a category, and so was placed in food (because it's a comestible for some reason?), even though its main (only?) use is making turpentine.

Describe the solution

JSON edits

Describe alternatives you've considered

None

Testing

Loaded a game, picked up some pine resin, looked at which category it was sorted into.

Pine resin previously didn't have a category, and so was placed in food
(because it's a comestible for some reason?), even though its main
(only?) use is making turpentine.
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Aug 7, 2022
@dseguin dseguin merged commit edbbdaa into CleverRaven:master Aug 7, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Aug 7, 2022
@pjf pjf deleted the pull/pine_category_fix branch August 8, 2022 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants