jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/328291 )
Change subject: Fix client-side $/sec calculation.
......................................................................
Fix client-side $/sec calculation.
Oops!
s/min/max
Change-Id: I49140f4cfa4e540bdd44eb42dbfbe13b42839bdf
---
M src/components/widgets/totals-earned-chart/totals-earned-chart.js
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Cdentinger: Looks good to me, approved
jenkins-bot: Verified
diff --git a/src/components/widgets/totals-earned-chart/totals-earned-chart.js
b/src/components/widgets/totals-earned-chart/totals-earned-chart.js
index f506be6..d45f3a0 100644
--- a/src/components/widgets/totals-earned-chart/totals-earned-chart.js
+++ b/src/components/widgets/totals-earned-chart/totals-earned-chart.js
@@ -145,7 +145,7 @@
day = el.day - offset,
hour = el.hour,
total = el.usd_total,
- seconds = Math.min( el.minutes
* 60, 60 ); // Don't divide by zero
+ seconds = Math.max( el.minutes
* 60, 60 ); // Don't divide by zero
params.sharedContext.dayObj[day][hour + 1] = {
total: total, count: el.donations };
params.sharedContext.secondsByHourDonationData[(day - 1) * 24 + hour + 1] =
el.usd_total / seconds;
--
To view, visit https://gerrit.wikimedia.org/r/328291
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I49140f4cfa4e540bdd44eb42dbfbe13b42839bdf
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/dash
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>
Gerrit-Reviewer: Cdentinger <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
