Skip to content

TemporaryBlindness StatusEffect#43705

Open
Pok27 wants to merge 1 commit intospace-wizards:masterfrom
Pok27:TemporaryBlindnessStatusEffect
Open

TemporaryBlindness StatusEffect#43705
Pok27 wants to merge 1 commit intospace-wizards:masterfrom
Pok27:TemporaryBlindnessStatusEffect

Conversation

@Pok27
Copy link
Copy Markdown
Contributor

@Pok27 Pok27 commented Apr 24, 2026

About the PR

Converts TemporaryBlindness into new status effects.

Why / Balance

#38575

Technical details

Media

Requirements

Breaking changes

The TemporaryBlindness status effect has been removed and replaced with StatusEffectTemporaryBlindness for the new status effect system.
The TemporaryBlindnessComponent has been removed and replaced with TemporaryBlindnessStatusEffectComponent

Changelog

@PJBot PJBot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/S Denotes a PR that changes 10-99 lines. labels Apr 24, 2026
@Pok27 Pok27 added P3: Standard Priority: Default priority for repository items. T: Cleanup Type: Code clean-up, without being a full refactor or feature D3: Low Difficulty: Some codebase knowledge required. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Apr 24, 2026
iaada
iaada approved these changes Apr 24, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@iaada iaada left a comment

Choose a reason for hiding this comment

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

Seems like a clean transition with this one. Also remove the .ftl entry for entity-effect-status-effect-TemporaryBlindness

@PJBot PJBot added the S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. label Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iaada iaada iaada approved these changes

@DrSmugleaf DrSmugleaf Awaiting requested review from DrSmugleaf DrSmugleaf is a code owner

@sowelipililimute sowelipililimute Awaiting requested review from sowelipililimute sowelipililimute is a code owner

Assignees

No one assigned

Labels

D3: Low Difficulty: Some codebase knowledge required. P3: Standard Priority: Default priority for repository items. S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/S Denotes a PR that changes 10-99 lines. T: Cleanup Type: Code clean-up, without being a full refactor or feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Pok27 @iaada @PJBot