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

#579 Updated for the stock counter to be updated on any hopper input #591

Merged
merged 4 commits into from May 14, 2024

Conversation

Johanmans10
Copy link
Contributor

@Johanmans10 Johanmans10 commented Apr 29, 2024

Closes #579


IssueHunt Summary

Referenced issues

This pull request has been submitted to:


Copy link
Member

@Phoenix616 Phoenix616 left a comment

Choose a reason for hiding this comment

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

Thank you for your pull request! I have some questions though about how you implemented it, see the comments.

pom.xml Outdated Show resolved Hide resolved
src/main/java/com/Acrobot/Breeze/Utils/InventoryUtil.java Outdated Show resolved Hide resolved
@Johanmans10
Copy link
Contributor Author

Should be all done. If you have any more questions or concerns please let me know.

Copy link
Member

@Phoenix616 Phoenix616 left a comment

Choose a reason for hiding this comment

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

Looks good now, thank you!

@Phoenix616 Phoenix616 merged commit af02eb3 into ChestShop-authors:master May 14, 2024
1 check passed
@Johanmans10 Johanmans10 deleted the stock-counter-update branch May 14, 2024 17:20
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

Successfully merging this pull request may close these issues.

Stock counter not updating when container is being filled by hopper
2 participants