Skip to content

Add readable placeholder exception to show instead of a java exception#4772

Merged
TarLaboratories merged 4 commits into1.20.1from
tar/add-no-target-exception
Apr 6, 2026
Merged

Add readable placeholder exception to show instead of a java exception#4772
TarLaboratories merged 4 commits into1.20.1from
tar/add-no-target-exception

Conversation

@TarLaboratories
Copy link
Copy Markdown
Contributor

What

Add NoTargetException to show when a placeholder that requires a target is executed without a target

Outcome

Resolves #4769

How Was This Tested

In-game

@TarLaboratories TarLaboratories requested a review from a team as a code owner April 6, 2026 16:35
@TarLaboratories TarLaboratories added type: bugfix General bug fixes Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Apr 6, 2026
@github-actions github-actions bot added 1.20.1 Tests: Passed Game Tests have passed on this PR labels Apr 6, 2026
Copy link
Copy Markdown
Contributor

@jurrejelle jurrejelle left a comment

Choose a reason for hiding this comment

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

I love love love!!! more clear exceptions and user-facing debugging help !!!!

Copy link
Copy Markdown
Contributor

@jurrejelle jurrejelle left a comment

Choose a reason for hiding this comment

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

bruh I clicked that button

@TarLaboratories TarLaboratories merged commit af86041 into 1.20.1 Apr 6, 2026
4 checks passed
@TarLaboratories TarLaboratories deleted the tar/add-no-target-exception branch April 6, 2026 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. Tests: Passed Game Tests have passed on this PR type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create's Linked Controller do not work in Central Monitor placeholders

2 participants