jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/394222 )
Change subject: Fix last data point day
......................................................................
Fix last data point day
Maybe the whole thing should live in amt/second chart?
Change-Id: I5998e5e1762a233e5f1cf63d8a71698e69f3dfbb
---
M src/components/widgets/totals-earned-chart/totals-earned-chart.js
1 file changed, 4 insertions(+), 3 deletions(-)
Approvals:
XenoRyet: 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 6ef43c5..286bd1e 100644
--- a/src/components/widgets/totals-earned-chart/totals-earned-chart.js
+++ b/src/components/widgets/totals-earned-chart/totals-earned-chart.js
@@ -134,7 +134,8 @@
hour,
total,
seconds,
- dataCount;
+ dataCount,
+ ms;
initializedSharedContext( params );
lastData = params.sharedContext.lastDataPoint;
@@ -173,8 +174,8 @@
}
if ( self.isCurrentYear() ) {
- // FIXME
- lastData.day = currentDate.getUTCDate();
+ ms = currentDate.getTime() -
self.campaign().getStartDate().getTime();
+ lastData.day = Math.floor( ms / ( 24 * 60 * 60
* 1000 ) ) + 1;
lastData.hour = currentDate.getUTCHours();
} else if ( dataCount > 0 ) {
lastData.day = data[ dataCount - 1 ].day -
offset;
--
To view, visit https://gerrit.wikimedia.org/r/394222
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5998e5e1762a233e5f1cf63d8a71698e69f3dfbb
Gerrit-PatchSet: 2
Gerrit-Project: wikimedia/fundraising/dash
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>
Gerrit-Reviewer: XenoRyet <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
