Skip to content

hotfix(server/systems): Fixed collision detection on ApplyMovement system - #239

Merged
nl1x merged 16 commits into
devfrom
hotfix/systems/apply-movements
Jan 18, 2026
Merged

hotfix(server/systems): Fixed collision detection on ApplyMovement system#239
nl1x merged 16 commits into
devfrom
hotfix/systems/apply-movements

Conversation

@nl1x

@nl1x nl1x commented Jan 18, 2026

Copy link
Copy Markdown
Collaborator

hotfix(server/systems): Fixed collision detection on ApplyMovement system

Issues closed

Closes #221

Summary

This pull request fix the collision detection on the ApplyMovement system of the server by handling horizontal and vertical movement separatly.
It also manage the movement of bullets and enemies.

Changes

Handles horizontal and vertical movement separatly.
Move enemies and bullets.
Detect when an enemy touch a player and kill the player and the enemy.
Detect when a bullet touch an enemy and kill the bullet and the enemy.

Breaking changes

  • This code contains breaking changes:

Checklist

  • I have renamed the first header of the template -_-
  • I have tested my modifications
  • I have done the relevant documentation
  • I have set the Reviewers & Assignees
  • I have set the correct Labels
  • I have set the rtype project
  • I have set Status / Priority / Size / Start date / End date
  • I have set the relevant Milestone

@nl1x nl1x self-assigned this Jan 18, 2026
@nl1x nl1x added this to the Final product milestone Jan 18, 2026
@nl1x
nl1x marked this pull request as ready for review January 18, 2026 01:01
Shuvlyy
Shuvlyy previously approved these changes Jan 18, 2026
Shuvlyy
Shuvlyy previously approved these changes Jan 18, 2026

@Shuvlyy Shuvlyy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Image

@PierreMarguerie
PierreMarguerie force-pushed the hotfix/systems/apply-movements branch from 4592090 to 66f7db6 Compare January 18, 2026 17:07
@nl1x
nl1x merged commit 7c5aec0 into dev Jan 18, 2026
13 checks passed
@nl1x
nl1x deleted the hotfix/systems/apply-movements branch January 18, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants