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

fix: correct score calculation in getDailyOverview function #25

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

cornedor
Copy link
Owner

Pull Request Description

Title: fix: correct score calculation in getDailyOverview function

Description:
This pull request addresses an issue with the score calculation in the getDailyOverview function. The logic has been updated to subtract the scores of the winning teams rather than adding them. This adjustment is crucial for ensuring that the scoring accurately reflects the intended game mechanics, leading to more precise game summaries.

Changes Made:

  • Modified the score calculation logic in the getDailyOverview function.
  • Ensured that the scoring now subtracts the winning teams' scores.

This change is expected to enhance the accuracy of the game summaries and improve the overall functionality of the scoring system.

Update the score calculation logic in the getDailyOverview function to 
subtract the scores of the winning teams instead of adding them. This 
change ensures that the scoring reflects the intended game mechanics 
and provides accurate game summaries.
Copy link

vercel bot commented Feb 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
creeplogger ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 6:51am

@cornedor cornedor merged commit da1834b into main Feb 14, 2025
1 of 2 checks passed
@cornedor cornedor deleted the cd-branch-9 branch February 14, 2025 06:50
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