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

Networking #4

Merged
merged 29 commits into from
Jan 7, 2025
Merged

Networking #4

merged 29 commits into from
Jan 7, 2025

Conversation

Jab125
Copy link
Contributor

@Jab125 Jab125 commented Jan 5, 2025 •
edited
Loading

TODO:

  • S2C Play
  • C2S Play
  • S2C Configuration
  • C2S Configuration
  • Configuration Events

@Jab125 Jab125 self-assigned this Jan 5, 2025
Jab125 added 23 commits January 6, 2025 15:19
@Jab125
reshuffle some classes
755a37e
@Jab125
reshuffle even more classes
e496dfb
@Jab125
add some javadocs
04ede78
@Jab125
Update NetworkingAbstractions.java
792986a
@Jab125
remove now unused class
ec1c56e
@Jab125
optimize imports
eb3e040
@Jab125
Update ClientboundPlay.java
a7f24b2
@Jab125
Create ClientboundConfiguration.java
5629757
@Jab125
Create ServerboundConfiguration.java
543e1a9
@Jab125
turns out you can't get the player during the configuration phase
664873e
@Jab125
Create ClientConfigurationEvents.java
22d683a
@Jab125
halp
e4f578c
@Jab125
implement InternalClientConfigurationEvents.START
fefacfb
@Jab125
Update InternalClientConfigurationEvents.java
1d6b1d4
@Jab125
add event for server configure
eac7053
@Jab125
add some javadoc
17fa695
@Jab125
add test configuration packet
12dcc5b
@Jab125
a few final touches
9919b09
@Jab125
fix neoforge oopsie
9d65a4f
@Jab125
add some documentation
da8485e
@Jab125
Update package-info.java
64080cd
@Jab125
Update ServerboundConfiguration.java
06bdc5e
@Jab125
log messages in chat
d997461
@Jab125 Jab125 requested a review from DexrnZacAttack January 6, 2025 07:54
@Jab125 Jab125 marked this pull request as ready for review January 6, 2025 07:54
@Jab125 Jab125 merged commit cd224d9 into master Jan 7, 2025
2 checks passed
@omo50 omo50 deleted the networking branch January 9, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DexrnZacAttack DexrnZacAttack Awaiting requested review from DexrnZacAttack

Assignees

@Jab125 Jab125

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Jab125