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

Update packets #3112

Merged
merged 14 commits into from
Mar 2, 2022
Merged

Update packets #3112

merged 14 commits into from
Mar 2, 2022

Conversation

4144
Copy link
Contributor

@4144 4144 commented Jan 24, 2022

Pull Request Prelude

  • I have followed [proper Hercules code styling][code].
  • I have read and understood the [contribution guidelines][cont] before making this PR.
  • I am aware that this PR may be closed if the above-mentioned criteria are not fulfilled.

Changes Proposed

Update shuffle packets and keys.
Update messages.
Add some missing packet versions and rename some existing packets to official names.

@4144 4144 force-pushed the updatepackets branch 3 times, most recently from 9e7630c to 2f8a9c5 Compare January 25, 2022 02:11
@Asheraf Asheraf added this to the Release v2022.02.02 milestone Jan 28, 2022
@MishimaHaruna
Copy link
Member

Some conflicts appeared since I merged #3107 and there was some overlap. Can you have a look at it @4144 ?

@4144
Copy link
Contributor Author

4144 commented Mar 2, 2022

yes i can
today was new client released, i will add new packets in next pull request...

@4144
Copy link
Contributor Author

4144 commented Mar 2, 2022

done

@MishimaHaruna MishimaHaruna merged commit af38409 into HerculesWS:master Mar 2, 2022
@4144 4144 deleted the updatepackets branch March 4, 2022 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants