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

Investigate Blitterblack Maze #363

Draft
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

pacampbell
Copy link
Collaborator

@pacampbell pacampbell commented Jun 26, 2024

Enabled warp to/from Cave Harbor to Bitterblack Maze.

Warning

Nothing works after entering. Need to support new packets.

Checklist:

  • The project compiles
  • The PR targets develop branch

Enabled warp from cave harbour to Bitterblack Maze.
- Added support for BattleContentGetContentStatusFromOm Req/Res
- Added partial decoding for S2CBattleContentInfoListRes
  - Need to implement decoding for final part (large list of numbers)
- Added new packet handler for BattleContentContentEntryHandler
- Continued to work decoding the contents of S2CBattleContentInfoListRes
  and S2CBattleContentContentEntryNtc. There seems to be an issue with
  the Unk4 and Unk5 fields in S2CBattleContentInfoListRes.
Fixed an issue where the fields for S2CBattleContentContentEntryNtc and
S2CBattleContentInfoListRes were encoded incorrectly.
Created BattleContentInstantClearInfoHandler to handle clear info
packets.
Added new packets C2SBattleContentPartyMemberInfoUpdateReq,
S2CBattleContentPartyMemberInfoUpdateRes and
S2CBattleContentPartyMemberInfoUpdateNtc. The client seems to be stuck
in a loop requesting this packet over and over so probably some more
fields need to be filled out.
Added some monsters to 2 maps. Used Freddy to warp to the maze from WDT.
- Fixed an issue where CDataBattleContentUnk4 didn't have an read/write
  serializer implementation.
- Played around with more values in the packet responses.
Added a enum which represents the current mode of the game.
Added more packets and annotated some fields based on some guesses.
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.

None yet

1 participant