Change 423144 had a related patch set uploaded (by Amitjoki; owner: Amitjoki):
[analytics/wikistats2@master] Add time period for maps and tops metrics

https://gerrit.wikimedia.org/r/423144

@Amitjoki I admire your enthusiasm, but whenever you take a task please share it in comments or claim to work on the task. I was about to submit a patch for the same.

@sahil505 don't know. I thought claiming task was akin to issues "Assigned" in Github. Usually if a person is assigned a issue, only his work will be considered right? Whatever additional done is supplementary, that was my rationale.

I started working long before and thought it would do any harm if I submitted my patch too, thinking only your patch would be considered. I just uploaded my patch because I didn't want to waste my changes.

If you've a better approach for solving the problem, please do upload them. I'll be more than happy to consider abandoning my changes if the need be :]

I'll also remember to comment if I am working on the same problem the next time :)

@Amitjoki I think you need to rework your patch a bit, please take a look at code on dashboard that already prints current month for top metrics.

https://github.com/wikimedia/analytics-wikistats2/blob/master/src/components/dashboard/MetricWidget.vue#L19

• data displayed on maps is only for the last month, your already have that data. See:

https://github.com/wikimedia/analytics-wikistats2/blob/master/src/components/detail/GraphPanel.vue#L161

• labels for months already exist, see vue filter:https://github.com/wikimedia/analytics-wikistats2/blob/master/src/main.js#L48

@Amitjoki No worries please go through with it. Although, I did take a look at your code and I don't think you need to add new methods for this issues, and also as far as I have understood you don't need to display the year as well. As @Nuria correctly mentioned above you can take an example from Dashboard list of metrics. While you are at it so take a look at https://github.com/wikimedia/analytics-wikistats2/blob/master/src/components/dashboard/MetricListWidget.vue#L38, this might reduce your code even more. Try to extract information from already computed functions. For better understanding of the description please also look at T187389, you might be able to relate to this description more. Thanks.

@sahil505 thanks for the information. I've reduced the code as per your @Nuria suggestions :]

I have modified commit message and submitted a patch. Can you add a screenshot to this ticket with changes we should see from your patch? Thanks

@Nuria thanks! Here is the screen recording https://drive.google.com/file/d/1obp9JGeBla1wPhoReoDsdxCnCmoyUiYG/view

I see where is the confusion, the legend should not be changed, it will not change per month rather header of page should change.

@Amitjoki Maybe we can find a better placement for displaying the month in top metric table. Dedicating an entire row just to display month feels like not the best design practice? What do you think? @Nuria What are your views?

Maybe this visual explanation makes more sense.

@Nuria yeah. That makes sense. I've updated my changes. Here's the preview: https://www.useloom.com/share/9dbf7ff90a8c48b499a1053ffb520e0b Thanks :]

Some suggestions I have, correct me if I'm wrong :)

• 'for February' makes more sense than 'on February', maybe?
• I feel like month has nothing to do with the page views hyperlink.
• Maybe just displaying February would suffice, because it automatically implies that the data is monthly

@sahil505 yes, I too felt weird about the first two points. We could display "February" in place of "Monthly", that would make more sense. Will be submitting a patch for the same :]

Here's the newer version which I think is more pragmatic: https://www.useloom.com/share/ea829701049a46ab886caf02bd7269f7

'for February' makes more sense than 'on February', maybe?

The copy used in the dashboard is "Countries with the most views for March" So "Pageviews by Country For March" makes sense.

I feel like month has nothing to do with the page views hyperlink.
Maybe just displaying February would suffice, because it automatically implies that the data is monthly

Agreed. The "monthly" after the new copy should be removed.

Please, @Amitjoki, do assign yourself to tasks you're working on.
Even if anyone can contribute to a currently open task, it's good to know that someone else is working on it.
This way you can decide to wait for their changes to appear on Gerrit, and then work on top of them; or rather work on another task.
It also avoids that 2 people work on the same thing twice, it reduces overall effort.
It really has no other meaning.

@sahil505 please feel free to submit your changes in a separate Gerrit patch, for this time.
Otherwise, you can work on top of Amil's change or work on another task.

@mforns : No worries at all :-) I'll find other tasks to work on.
Removing myself from 'Assigned To'.

@sahil505 and @mforns sorry! I'll make sure to let it be known if I am working on a task. Sorry for the inconvenience.

Change 423144 merged by Mforns:
[analytics/wikistats2@master] Label map and top metrics with the month they belong to

https://gerrit.wikimedia.org/r/423144