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

Fire damage is inconsistent #418

Open
cs716 opened this issue Jun 7, 2024 · 2 comments
Open

Fire damage is inconsistent #418

cs716 opened this issue Jun 7, 2024 · 2 comments
Assignees
@DrewBritt
Labels
bug Something is not working as intended needs tested / review The issue is potentially fixed but requires more testing or a review

Comments

@cs716
Copy link
Member

cs716 commented Jun 7, 2024

Fire will sometimes do zero damage
@cs716 cs716 added the bug Something is not working as intended label Jun 7, 2024
@DrewBritt DrewBritt self-assigned this Jun 19, 2024
DrewBritt added a commit that referenced this issue Jun 19, 2024
@DrewBritt
Other players check for fire damage (#418)
54982ea
@DrewBritt DrewBritt added the needs tested / review The issue is potentially fixed but requires more testing or a review label Jun 19, 2024
@DrewBritt
Copy link
Member

Tested this with both a laptop and a friend and it seemed fine, but we'll know for sure next playtest with edge scenarios.

@DrewBritt DrewBritt removed the needs tested / review The issue is potentially fixed but requires more testing or a review label Jun 24, 2024
@DrewBritt DrewBritt added the needs tested / review The issue is potentially fixed but requires more testing or a review label Jul 5, 2024
@DrewBritt
Copy link
Member

Have pushed a new fix that I'm hopeful for; proxy clients should correctly check for fire destruction (previously it appeared to only work in host -> client and client -> host scenarios).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DrewBritt DrewBritt

Labels
bug Something is not working as intended needs tested / review The issue is potentially fixed but requires more testing or a review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cs716 @DrewBritt