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
Implemented campfires #596
base: master
Are you sure you want to change the base?
Conversation
* Campfires engite when flaming entity moves accross
Implemented Splash Potions having there own interface
Campfires now throw out cooked food Implemented Campfire Sounds Known Bugs (Campfires bug out when you click them, something to do with the Activate method)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks alright, needs periods and commas in some of your comments.
* Disabled campfire vertical stacking
Co-authored-by: DaPigGuy <mcpepig123@gmail.com>
Co-authored-by: DaPigGuy <mcpepig123@gmail.com>
… campfire and begin eating their food.
Make sure you fix the import cycle. |
…ampfire # Conflicts: # server/entity/splashable.go
This reverts commit 2277131.
Co-authored-by: DaPigGuy <mcpepig123@gmail.com>
Co-authored-by: DaPigGuy <mcpepig123@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine mostly! I've added a couple comments and questions.
@@ -141,6 +141,7 @@ func (h *InventoryTransactionHandler) handleUseItemTransaction(data *protocol.Us | |||
s.c.BreakBlock(pos) | |||
case protocol.UseItemActionClickBlock: | |||
s.c.UseItemOnBlock(pos, cube.Face(data.BlockFace), vec32To64(data.ClickedPosition)) | |||
s.c.UseItem() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure this is a correct change? This sounds like a change that has quite some implications.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tal recommened I make that change. This was added in order to fix every raw food type that interacted with the campfire from instantly getting placed. The ideal behavior here is for each click to add a signular raw food type onto the campfire. I'm not really sure how to attain this behaviour.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JustTalDevelops could you elaborate on this? I'm not entirely sure what the issue is here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any news?
b4b1fbc
to
6016986
Compare
Campfires!