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

Implement a rudimentary packet log in-game. #874

Closed
Measurity opened this issue Jan 4, 2020 · 3 comments · Fixed by #1038
Closed

Implement a rudimentary packet log in-game. #874

Measurity opened this issue Jan 4, 2020 · 3 comments · Fixed by #1038
Labels
Complexity: easy Suitable for contributors new to Nitrox

Comments

@Measurity
Copy link
Collaborator

Measurity commented Jan 4, 2020 •
edited
Loading

There’s a stub for this already in Nitrox but it has no implementation. Ideally it should show how many packets of X have been sent/received.

Getting started

  • PacketReceiver class for detecting what kind of packets are received.
  • MultiplayerSessionManager class for detecting what kind of packets are send.
  • NetworkDebugger class should be where a new gui is rendered displaying the types of packets sent/received.

Nice-to-have

  • Being able to view what a packet contained.
@Measurity Measurity added Complexity: easy Suitable for contributors new to Nitrox help wanted labels Jan 4, 2020
@j657
Copy link

j657 commented Jan 9, 2020

Cool

@ghost
Copy link

ghost commented Jan 14, 2020

I'll check that

@Measurity
Copy link
Collaborator Author

@iCleeem If you are working on this please join our discord and post it on the jr-alterra channel as well. Tell the moderators you're interested in helping out if you aren't jr-alterra yet.

@Measurity Measurity assigned ghost Jan 22, 2020
@ghost ghost removed their assignment Feb 8, 2020
@EtienneDx EtienneDx mentioned this issue Apr 12, 2020
Merged
@Measurity Measurity linked a pull request Apr 14, 2020 that will close this issue
Feature packet log #1038
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Complexity: easy Suitable for contributors new to Nitrox
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature packet log EtienneDx/Nitrox
3 participants
@Measurity @j657 and others