Skip to content

fix: correct arcade disasters ratios#843

Open
ugcodrr wants to merge 3 commits into
mainfrom
fix/arcade-disasters-ratios
Open

fix: correct arcade disasters ratios#843
ugcodrr wants to merge 3 commits into
mainfrom
fix/arcade-disasters-ratios

Conversation

@ugcodrr
Copy link
Copy Markdown
Member

@ugcodrr ugcodrr commented May 12, 2026

Summary

  • derive Disasters losses from total deaths when the API does not provide a losses value
  • keep /ratios arcade on Disasters stats for the Disasters overall submode instead of falling back to Arcade-level stats

Testing

  • pnpm --filter @statsify/schemas test:types
  • pnpm --filter discord-bot test:types

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
statsify Ready Ready Preview, Comment May 12, 2026 7:20pm

ugcodrr
ugcodrr commented May 12, 2026
View reviewed changes
Comment thread packages/schemas/src/player/gamemodes/arcade/mode.ts Outdated
@ugcodrr ugcodrr requested review from Copilot and jacobk999 May 28, 2026 20:20
Copilot AI reviewed May 30, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@jacobk999 jacobk999 Awaiting requested review from jacobk999

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

discord-bot @statsify/schemas

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ugcodrr