Skip to content

Consort, RK, Hand, Adventurer bugfixes#5744

Merged
KevinfromHP merged 5 commits intoMonkestation:mainfrom
Telvayne:rk-hand-consort-fixes-and-triumph-buy-restriction
Mar 25, 2026
Merged

Consort, RK, Hand, Adventurer bugfixes#5744
KevinfromHP merged 5 commits intoMonkestation:mainfrom
Telvayne:rk-hand-consort-fixes-and-triumph-buy-restriction

Conversation

@Telvayne
Copy link

@Telvayne Telvayne commented Mar 20, 2026

About The Pull Request

This PR has been adjusted from original purpose. It is not simply bugfixes for Hand, Consort, RK, and Adventurer. Keys should spawn more reliably for triumph purchasers. Adventurers should no longer the the double-torch bug. There is language in this PR about restricting triumph purchases, but Kevin fixed that in another PR, so it is no longer needed here.

Why It's Good For The Game

A good deal of administrative overhead with the adv. class triumph buy is fixed with this. Hand and RK key issues will no longer occur. RKs WILL spawn with their weapon in their hand; they can fix this pretty easily by just putting their keys in their belt.

Changelog

🆑
qol: Fixed RK sprite on the character selection screen
qol: Adventurers spawn with a lit torch, and not two torches
balance: Fixed RK stats and skills to be part of the adv class so they don't double-dip on adv class triumph buy
fix: Moved Hand, RK keys to belt.
/:cl:

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

@github-actions github-actions bot added Fix This PR is for fixing bugs Quality of Life The hardest thing to decide is if this is NOT a balance change Removal What used to be there is no longer there. How sad. labels Mar 20, 2026
Copy link
Collaborator

@KevinfromHP KevinfromHP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i already handled the stuff on the triumph buy. no need to touch that.

@Telvayne
Copy link
Author

i already handled the stuff on the triumph buy. no need to touch that.

Sounds good! I'll go in and revert that part, but i'll keep the bugfixes for the three classes.

@Telvayne
Copy link
Author

This one is ready to go! It's ended up being just bug fixes for adv class triumph buy to ease admin overview. It's removed the nonsense double-stat double-skill stacking from RK triumph buy, removed the double torch from some adventurer classes, and secured hand and RK keys in a safe place so they don't get deleted

@Telvayne Telvayne changed the title Restrict Adv Class Triumph Purchase - No Consort, RK, or Hand. Also bug fixes Consort, RK, Hand, Adventurer bugfixes Mar 21, 2026
@Telvayne
Copy link
Author

Little update, I had a random thought and checked it; the RK subclasses never equip anything to wrist, so that's where the keyring is now.

@github-actions github-actions bot added the MERGE CONFLICT This PR has conflicting changes with the main branch and can't be merged label Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix This PR is for fixing bugs Quality of Life The hardest thing to decide is if this is NOT a balance change Removal What used to be there is no longer there. How sad. Test Merge Actively test-merged or relating to test merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants