872198 - Add view count to questions listing page

Status: RESOLVED FIXED

(support.mozilla.org :: Questions, task, P2)

Description

[Ricky Rosario [:rrosario, :r1cky]]

We need to add the view counts to the question listing page. I would love to remove the "has this problem" count and just leave views and "has this problem this week". The total "has this problem" count is one of the things that is hurting our performance on this page.

From Bug 857534:
I wasn't exactly sure where to add it and I want to spend some time making sure it doesnt kill our performance more on that page. Also, need to figure out what to do in the case of new questions that don't have a view count until the cron jobs overnight.

Comment 1

[Kadir Topal [:atopal]]

Bug 857534 already states what to do with read counts: They would only be displayed on the thread itself, but not on the thread listing page. Since we already have it on the individual thread we can just go ahead and remove it from the thread listing page. There was no concern the last time I proposed it in the platform meeting.


Essentially, let's swap "me toos" for views on the thread listing page.

For the case where we don't have any views yet we can say so: "NA." Unless there is a better way to handle it.

Comment 2

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to Kadir Topal [:atopal] from comment #1)

OK, so we won't show votes on the listing page anymore? Is this good: http://cl.ly/image/3v2x1F2R3I43 ?

Comment 3

[Kadir Topal [:atopal]]

To clarify we are swapping the votes for views. But we should keep the "views this week" feature. Otherwise sorting by views will become completely meaningless within weeks.

Comment 4

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to Kadir Topal [:atopal] from comment #3)
> To clarify we are swapping the votes for views. But we should keep the
> "views this week" feature. Otherwise sorting by views will become completely
> meaningless within weeks.

views, votes, so confusing! This is what I think you meant:

To clarify we are swapping the total votes for views. But we should keep the
"votes this week" feature. Otherwise sorting by votes will become completely
meaningless within weeks.

Comment 5

[Kadir Topal [:atopal]]

Heh, it's confusing indeed, but I really meant "views this week". The total view count is great for motivation, but if you want to do anything meaningful with it, you have to use the bounded version, eg. weekly data.

Comment 6

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to Kadir Topal [:atopal] from comment #5)
> Heh, it's confusing indeed, but I really meant "views this week". The total
> view count is great for motivation, but if you want to do anything
> meaningful with it, you have to use the bounded version, eg. weekly data.

well, we didn't implement views per week. Only total views. Why wasn't that spec'd in Bug 857534 ?

Comment 4 stands.

Comment 7

[Kadir Topal [:atopal]]

It was spec'd in bug 857534:

> * We should get uniques for last 24 hours and add to our current value.
> * We should get uniques for the last week to show on the thread listing page.

Comment 8

[Kadir Topal [:atopal]]

Aside, I'm not sure what we'll accomplish with showing only total views. It's okay for motivation, but sorting by them would be quite meaningless.

Comment 9

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to Kadir Topal [:atopal] from comment #7)
> > * We should get uniques for last 24 hours and add to our current value.
> > * We should get uniques for the last week to show on the thread listing page.

Gah. Buried in the comments. I didn't pay attention because that first one is an invalid suggestion of an implementation detail. The only screenshot provided (http://note.io/10pDZKv) didn't suggest having views on the question listing page.

Anyway, if weekly views are really important lets open a bug.

I think it would be better to add filtering by date to the questions list and then we wouldnt need that at all.

Comment 10

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to Kadir Topal [:atopal] from comment #8)
> Aside, I'm not sure what we'll accomplish with showing only total views.
> It's okay for motivation, but sorting by them would be quite meaningless.

Yeah but sorting wasnt ever requested :(. The user story doesn't suggest sorting is important:

User Story: As a contributor I'd like to see how often threads get read. Knowing that a thread has been read 3000 times this month means I'm more motivated to get to a solution than when it's only being read by one person.

Comment 11

[Ricky Rosario [:rrosario, :r1cky]]

Making this 1pt to change all time votes for all time views on the question listing page.

Comment 12

[Ricky Rosario [:rrosario, :r1cky]]

In a pull request: https://github.com/mozilla/kitsune/pull/1400

Comment 13

[Ricky Rosario [:rrosario, :r1cky]]

Landed on master:
https://github.com/mozilla/kitsune/commit/aedd2c318c58047d053340ef589b2628ee3b871e

And deployed to prod.