Skip to content

SERVER-NETWORK/CLIENT-NETWORK: Fix wrong id sent to the order side of…#83

Merged
guillaumeAbel merged 2 commits intodevfrom
fix/wrong-id-for-enemy-death
Oct 12, 2023
Merged

SERVER-NETWORK/CLIENT-NETWORK: Fix wrong id sent to the order side of…#83
guillaumeAbel merged 2 commits intodevfrom
fix/wrong-id-for-enemy-death

Conversation

@romainpanno
Copy link
Copy Markdown
Collaborator

@romainpanno romainpanno commented Oct 12, 2023

… the dead enemy

Add createEntity also for server in initEnemy

PATCH

  • Please check if the PR fulfills these requirements
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Norm of the code has been respected
  • In what state is your pull request?
  • Ready to merge / Waiting a reviwer to see my work.
  • Work In Progress (WIP) / My work is not finish but i want daily reviews. (Draft)
  • CI Review / I want to see what the CI thinks of my work.
  • Need feedback / Just to have feedback on what i produced. (May not be merged)
  • What kind of change does this PR introduce? (You can choose multiple)
  • Bug fix
  • Feature request
  • New / Updated documentation
  • Testing CI ( Make the pull request in draft mode)
  • What is the current behavior? (link an issue based on the kind of change this pr introduce)

  • What is the new behavior (if this is a feature change)?

  • Other information:

… the dead enemy

Add createEntity also for server in initEnemy

PATCH
@romainpanno romainpanno added the bug Something isn't working label Oct 12, 2023
@romainpanno romainpanno self-assigned this Oct 12, 2023
@guillaumeAbel guillaumeAbel merged commit 9043d93 into dev Oct 12, 2023
@Saverio976 Saverio976 deleted the fix/wrong-id-for-enemy-death branch October 17, 2023 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants