Skip to content

[1.21] Fixed & extended MishapInternalException - #1259

Closed
YukkuriC wants to merge 1 commit into
FallingColors:1.21from
YukkuriC:feature/mishap-error
Closed

[1.21] Fixed & extended MishapInternalException#1259
YukkuriC wants to merge 1 commit into
FallingColors:1.21from
YukkuriC:feature/mishap-error

Conversation

@YukkuriC

Copy link
Copy Markdown
Contributor

fixes #1229
image

and adds detailed stacktrace (hover + click copy) for its debugging purpose
image

@object-Object object-Object left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, but is there a particular reason this has to be for 1.21, or could it be merged into main instead so we can have this on 1.20 as well?

@YukkuriC

Copy link
Copy Markdown
Contributor Author

LGTM, but is there a particular reason this has to be for 1.21, or could it be merged into main instead so we can have this on 1.20 as well?

yes it could, the primary reason for this PR is the kicking issue, and the stacktrace part is "we have to do manual toString anyway so why not" :3

@YukkuriC YukkuriC changed the title Fixed & extended MishapInternalException [1.21] Fixed & extended MishapInternalException Aug 16, 2026
@object-Object

Copy link
Copy Markdown
Member

Do we need to also merge this one, or is it ok to close this and merge #1268 into 1.21 in a main to next bundle?

@YukkuriC

Copy link
Copy Markdown
Contributor Author

Do we need to also merge this one, or is it ok to close this and merge #1268 into 1.21 in a main to next bundle?

both ok for me :3

@github-project-automation github-project-automation Bot moved this from 📋 Backlog to ✅ Done in Hex Casting Aug 16, 2026
hexxy-weblate pushed a commit to hexxy-weblate/HexMod that referenced this pull request Aug 16, 2026
FallingColors#1259, but for branch main

> ~~fixes FallingColors#1229~~
> ~~and~~ adds detailed stacktrace (hover + click copy) for its
debugging purpose
> <img width="474" height="182" alt="image"
src="https://github.com/user-attachments/assets/67df25c7-a6a0-462f-9c92-900dcfe55fd7"
/>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants