Issue Description:
“Melee” action (in “kill_feed” event) has two different meanings (it can be either “kill” or “knockdown”). To handle it, we need to process two different events to find out how to count the number of kills and knockdowns for an attacker. For example:
Knockdown with a melee attack:
{"name":"kill_feed","data":"{\r\n \"attackerName\": \"MrNooblik\",\r\n \"victimName\": \"Sagor_77rus\",\r\n \"weaponName\": \"Melee\",\r\n \"action\": \"Melee\"\r\n}"} {"name":"knockdown","data":"{\r\n \"victimName\": \"Sagor_77rus\"\r\n}"}
Kill with a melee attack:
{"name":"kill_feed","data":"{\r\n \"attackerName\": \"MrNooblik\",\r\n \"victimName\": \"Sagor_77rus\",\r\n \"weaponName\": \"Melee\",\r\n \"action\": \"Melee\"\r\n}"} {"name":"kill","data":"{\r\n \"victimName\": \"Sagor_77rus\"\r\n}"}
Steps to reproduce:
- Hit an alive enemy with a melee attack.
- The enemy will be knocked down but “kill_feed” says that it’s just a melee attack.
- Hit the knocked down enemy with a melee attack.
- The enemy will be killed but “kill_feed” still says that it’s just a melee attack.
To count the exact number of knockdowns and kills, we have two link two different events – it leads to some issues in the app.
Suggestion:
Replace “Melee” with “Melee Knockdown” and “Melee Kill” for “knockdown” and “kill” respectively instead of sending two different events. For example:
Knockdown with a melee attack:
{"name":"kill_feed","data":"{\r\n \"attackerName\": \"MrNooblik\",\r\n \"victimName\": \"Sagor_77rus\",\r\n \"weaponName\": \"Melee\",\r\n \"action\": \"Melee Knockdown\"\r\n}"}
Kill with a melee attack:
{"name":"kill_feed","data":"{\r\n \"attackerName\": \"MrNooblik\",\r\n \"victimName\": \"Sagor_77rus\",\r\n \"weaponName\": \"Melee\",\r\n \"action\": \"Melee Kill\"\r\n}"}
Impact for my app: high