Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Roshan kill message to include Radiant/Dire percentages #453

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Geczy
Copy link
Member

@Geczy Geczy commented Aug 24, 2024

Update the generateRoshanMessage function to include Radiant/Dire percentages.

  • Add Radiant/Dire percentage calculation:
    • Add radiantPercentage and direPercentage properties to the RoshRes interface.
    • Create calculateRadiantDirePercentages function to calculate Radiant/Dire percentages based on minS and maxS.
    • Update generateRoshanMessage function to call calculateRadiantDirePercentages and include the percentages in the message.
  • Update Roshan event handler:
    • Add radiantPercentage and direPercentage properties to the res object in the Roshan event handler.
    • Update Redis storage to include the new properties.

#416


For more details, open the Copilot Workspace session.

Update the `generateRoshanMessage` function to include Radiant/Dire percentages.

* **Add Radiant/Dire percentage calculation:**
  * Add `radiantPercentage` and `direPercentage` properties to the `RoshRes` interface.
  * Create `calculateRadiantDirePercentages` function to calculate Radiant/Dire percentages based on `minS` and `maxS`.
  * Update `generateRoshanMessage` function to call `calculateRadiantDirePercentages` and include the percentages in the message.
* **Update Roshan event handler:**
  * Add `radiantPercentage` and `direPercentage` properties to the `res` object in the Roshan event handler.
  * Update Redis storage to include the new properties.

#416

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/dotabod/backend?shareId=XXXX-XXXX-XXXX-XXXX).
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.

1 participant