Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Condense list of error users by removing inactive users #76

Open
rossjrw opened this issue Aug 7, 2023 · 0 comments
Open

Condense list of error users by removing inactive users #76

rossjrw opened this issue Aug 7, 2023 · 0 comments
Labels
feature A new feature

Comments

@rossjrw
Copy link
Member

rossjrw commented Aug 7, 2023

I currently publicly list error users as a way of making them visible to myself and other users who may then go on to let them know they have an error.

However, a significant portion of these users are inactive, estimated based on their account history - e.g. an inactive SCP user might have a sandbox page, a forum thread about it, their notification config, and no activity since those pages were created. There's not much point listing these users as they're unlikely to ever notice that they have an error and it makes active users in the list more difficult to notice.

I should remove these users from the list. I could maybe list them separately in a collapsible.

From the site side, this is easy - filter the first list for those lacking a tag that indicates the user is inactive, and filter for that tag's presence in the second list.

From the backend side, adding that tag is easy - I already do this for some situations:

Programmatically determining exactly when a user is considered inactive may take some nuance. I might consider conditions like:

  • Number of waiting notifications > 0
    • Difficulty: trivial
    • This metric might be a red herring. After Filter users before iterating for notification #59 nothing is evaluated for a user who doesn't have any waiting notifications, so they won't even be marked as an error user (other than users marked before then).
  • Recent forum activity
    • Difficulty: medium - no current implementation, but I have the data already
    • Note that I would still have this data even after Purge old data #65 as I need to track forum posts created by subscribed users regardless
    • I don't think this is fair - an inactive user wanting notifications from old posts is a perfectly valid use case.
      • Note to self: this change doesn't prevent that user from getting notifications, just reduces the visibility of their error report. It's not that big a deal
  • Some metric for 'has ever been regularly active'
    • E.g. a broad filter could be 'has ever received 5 replies to their posts/threads'.

Current best filter I can think of:

  • has made 5 posts OR has recieved 5 replies OR subscribed < 3mo ago OR last posted < 1mo ago
  • That filters out users who have been active at some point (either by making lots of posts or making few well-recieved posts), while not filtering out new users who only became active in the last 3 months, and not filtering out users who are provably active right now
  • Account creation date doesn't matter, history starts from when they subscribed to notifier
@rossjrw rossjrw added the feature A new feature label Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature
Projects
None yet
Development

No branches or pull requests

1 participant