nextcloud / nextcloud/user_retention

`user_created_at` not set for all users

Open
#847 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
PHP
Stars
11
Forks
8
Avg merge
11h 48m
Merged PRs (30d)
1

Description

How to use GitHub
  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

While checking, if the relevant Nextcloud accounts are correctly disabled based on the policy, I was reading through https://github.com/nextcloud/user_retention/issues/799. While we were aware, that the app can only start counting the days from the day it was installed, I didn't know about the user_created_at user setting mentioned in https://github.com/nextcloud/user_retention/issues/799#issuecomment-2654004910.
Manipulating manually that setting would even allow to make the app work retroactively, even if it was just installed. Great info!

However, while checking the setting of a few accounts, it seems it was not set for all users.
For example, my own account doesn't have that setting set. I assume it is because of it either being a Nextcloud admin or in one of the excluded groups.

While it does make sense to exclude certain accounts from all effects of the app, my account might not be and admin or be part of excluded groups in the future. I expect my account to be treated the same, when that happens. As the setting is not set, this seems to not be the case.

Contributor guide

Open the contributing guide

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 with the behavior described in issue #799 and its referenced comment about the user_created_at setting, then trace how this app handles accounts that are admins or belong to excluded groups. Identify the existing entry point and tests for setting or skipping this user setting. Done means eligible users consistently receive the setting and remain correctly handled if their exclusion status changes.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
40/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.