Skip to content

dev-yoyo#3

Closed
Itz-Yoyo wants to merge 12 commits into
masterfrom
dev-yoyo
Closed

dev-yoyo#3
Itz-Yoyo wants to merge 12 commits into
masterfrom
dev-yoyo

Conversation

@Itz-Yoyo

Copy link
Copy Markdown
Collaborator

new JockeyRole
騎士職業 from dtc2
please check

@LilyKensa

Copy link
Copy Markdown
Owner
  • Bad code style
  • Wrong potion duration, it's measured in ticks
  • Missing english translation for the role
  • Inconsistent role description format

- fix bad description
- fix potion duration format
- add English translation
- code style is good :)
@Itz-Yoyo

Copy link
Copy Markdown
Collaborator Author

done

# Conflicts:
#	src/main/java/dev/huey/destroyTheCore/managers/RolesManager.java
@Itz-Yoyo

Copy link
Copy Markdown
Collaborator Author

;-;

- fix JockeyRole dialogue style
- add Noble role
@Itz-Yoyo

Itz-Yoyo commented Jan 19, 2026

Copy link
Copy Markdown
Collaborator Author
Jockey drawio

How JockeyRole work. for reviewing

@Itz-Yoyo

Itz-Yoyo commented Jan 19, 2026

Copy link
Copy Markdown
Collaborator Author

4 new issues found by bensonshieh and ItzYoyo

  • [Solved] horse inventory bug
  • [Not Affect] getNearbyEntity queue randomly
  • [Solved] horse access problem
  • [Solved] being able to steal other player's horse inventory
  • [Solved] horse death drop saddle and horse armor

@Itz-Yoyo

Itz-Yoyo commented Jan 19, 2026

Copy link
Copy Markdown
Collaborator Author

5 new issues found by ItzYoyo and bensonhsieh

  • [Unsolved] noble can't die (onPlayerDeath handling problem)
  • [Unsolved] player death give noble 2 gold instead of 1 (first time death is working but not working after)
  • [Unsolved] noble helmet is not cleaned after switching role -> setTrash()
  • [Not Affect] switching role while on horse will evade detection, you can ride horse without being jockey
  • [Unsolved] player is once noble cannot die (由儉入奢易 由奢入儉難)
    Imma go to [sleep]
    @LilyKensa pls explain how handlePlayerDeath work thx
NobleRole drawio

@LilyKensa

LilyKensa commented Mar 9, 2026

Copy link
Copy Markdown
Owner

This branch is too unclear. I can help with the debugging IRL when you have free time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants