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.

Sign up for GitHub

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

Jump to bottom

Log furnace inventory actions #3141

Merged
merged 1 commit into from
Jul 27, 2024
Merged

Log furnace inventory actions #3141

merged 1 commit into from
Jul 27, 2024

Conversation

Emojigit
Copy link
Contributor

This PR adds inventory logging functions to furnaces. This PR is ready for review.

appgurueu
appgurueu approved these changes Jul 25, 2024
View reviewed changes
Copy link
Contributor

@appgurueu appgurueu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code checks out, makes sense for consistency with other nodes with inventories.

@sfan5 sfan5 merged commit 9cb57ae into minetest:master Jul 27, 2024
2 checks passed
MoNTE48 pushed a commit to MoNTE48/minetest_game that referenced this pull request Aug 17, 2024
@Emojigit @MoNTE48
Log furnace inventory actions (minetest#3141)
f0c2f6f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@appgurueu appgurueu appgurueu approved these changes

Assignees
No one assigned
Labels
One approval
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Emojigit @appgurueu @sfan5