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

Spectator Healthbar Shows #49

Closed
Altanis opened this issue Nov 1, 2022 · 1 comment · Fixed by #55
Closed

Spectator Healthbar Shows #49

Altanis opened this issue Nov 1, 2022 · 1 comment · Fixed by #55
Labels
inaccuracy Tested stat related inconsistencies between a real diep server this replica

Comments

@Altanis
Copy link
Contributor

Altanis commented Nov 1, 2022

What is inconsistent
The healthbar of someone using the spectator tank is visible.

Expected behavior
A spectator should not have a healthbar.

Screenshots
image

@Altanis Altanis added the inaccuracy Tested stat related inconsistencies between a real diep server this replica label Nov 1, 2022
@ABCxFF
Copy link
Owner

ABCxFF commented Nov 1, 2022

Further, the spectator shouldn't be sent to the client at all.

ABCxFF added a commit that referenced this issue Nov 11, 2022
@ABCxFF ABCxFF linked a pull request Nov 11, 2022 that will close this issue
1 task
ABCxFF added a commit that referenced this issue Nov 13, 2022
* patch: #2

* patch #44

* patch #49 - make specs invisible

* patch #50 -

* use buffers for writing

* Revert 00 chunking

* also patch #4

* finish #2, using fin
Camvanaa pushed a commit to Camvanaa/diepcustom that referenced this issue Jan 12, 2025
…xFF#55)

* patch: ABCxFF#2

* patch ABCxFF#44

* patch ABCxFF#49 - make specs invisible

* patch ABCxFF#50 -

* use buffers for writing

* Revert 00 chunking

* also patch ABCxFF#4

* finish ABCxFF#2, using fin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inaccuracy Tested stat related inconsistencies between a real diep server this replica
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants