NodeBB / NodeBB/nodebb-widget-essentials

Popular topics - post date vs last update

Open
#36 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Go Template
Stars
20
Forks
34
PR merge metrics
No merged PRs in 30d

Description

I notice the popular topics widget uses the topic creation date rather than the last update time.

In other words, you could have the duration set to "month" and there could be a topic that gets commented on / viewed twice as much as any other thread every single day, but because it was created longer than a month ago, it's not shown in the popular topics list.

This seems wrong to me. If the topic is still very active (i.e. commented on / viewed the most in the last month), it should appear in the list.

I can't really think of a good use case for the way it's currently done but if there is one, maybe this could be added as a toggle option so people can still use the creation date if they want to...

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the popular topics widget entry point and the logic that applies its duration setting. Compare the current topic-creation-date behavior with the requested recent-activity behavior, then verify that active older topics can appear in the selected period without breaking the existing list.

Written by the indexing model from the issue text.

Assessment

Domain
frontend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.