Skip to content

Disallow NPCs with 0 health from spawning#273

Open
Delta-07 wants to merge 16 commits into
masterfrom
update-health
Open

Disallow NPCs with 0 health from spawning#273
Delta-07 wants to merge 16 commits into
masterfrom
update-health

Conversation

@Delta-07

@Delta-07 Delta-07 commented Jul 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • Renamed is_spawned to force_despawn, introduced is_spawned().
  • Disallow NPCs with 0 health from spawning.
  • Refresh character health upon tickable step summons.
  • Moved removal_mode out of our TickableStep config and into our NPC config.

@Delta-07 Delta-07 requested a review from soir20 July 2, 2026 22:42
soir20
soir20 reviewed Jul 2, 2026
View reviewed changes
Comment thread src/game_server/handlers/character.rs Outdated
@Delta-07 Delta-07 requested a review from soir20 July 5, 2026 03:41
@Delta-07 Delta-07 changed the title Despawn NPCs if their health is empty on spawn Disallow NPCs with 0 health from spawning Jul 5, 2026
soir20
soir20 reviewed Jul 7, 2026
View reviewed changes
Comment thread src/game_server/handlers/character.rs Outdated
soir20
soir20 reviewed Jul 7, 2026
View reviewed changes
Comment thread src/game_server/handlers/character.rs Outdated
Delta-07 and others added 2 commits July 6, 2026 21:58
@Delta-07 @soir20
Prevent possible overflow on hp delta calc
79ff0c5 
Co-authored-by: soir20 <71418127+soir20@users.noreply.github.com>
@Delta-07
reset spawn stats in single method
5d791cf
@Delta-07 Delta-07 requested a review from soir20 July 7, 2026 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@soir20 soir20 Awaiting requested review from soir20

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Delta-07 @soir20