Skip to content

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented Nov 10, 2025

  • Resolves: #

Summary

Adds a remember me checkbox to the login form.

Checklist

@come-nc come-nc added this to the Nextcloud 33 milestone Nov 10, 2025
@come-nc come-nc self-assigned this Nov 10, 2025
@come-nc come-nc requested review from a team as code owners November 10, 2025 13:12
@come-nc come-nc added the 3. to review Waiting for reviews label Nov 10, 2025
@come-nc come-nc requested review from icewind1991, nfebe, salmart-dev, sorbaugh and susnux and removed request for a team November 10, 2025 13:12
@come-nc
Copy link
Contributor Author

come-nc commented Nov 10, 2025

/compile /

ITrustedDomainHelper $trustedDomainHelper,
string $user = '',
string $password = '',
bool $rememberme = false,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
bool $rememberme = false,
bool $remember_me = false,

for consistency with the other parameters? not sure

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That would force me to rename all the camelCase to rememberMe then for consistency 😒

@come-nc come-nc force-pushed the feat/add-rememberme-checkbox branch from 5bef2d7 to 6eb9804 Compare November 10, 2025 14:30
@come-nc
Copy link
Contributor Author

come-nc commented Nov 13, 2025

/compile rebase

@nextcloud-command nextcloud-command force-pushed the feat/add-rememberme-checkbox branch from 6eb9804 to bb22e56 Compare November 13, 2025 09:31
Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code look ok :)

@sorbaugh
Copy link
Contributor

/backport to stable32

@sorbaugh
Copy link
Contributor

/backport to stable31

@come-nc
Copy link
Contributor Author

come-nc commented Nov 13, 2025

/compile rebase

come-nc and others added 4 commits November 13, 2025 13:25
Only present if allowed by configuration.

Signed-off-by: Côme Chilliet <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@nextcloud-command nextcloud-command force-pushed the feat/add-rememberme-checkbox branch from 9a7744d to 907fb19 Compare November 13, 2025 13:32
@come-nc come-nc enabled auto-merge November 13, 2025 14:55
@sorbaugh sorbaugh disabled auto-merge November 13, 2025 15:53
@sorbaugh sorbaugh merged commit 1498b5d into master Nov 13, 2025
223 of 245 checks passed
@sorbaugh sorbaugh deleted the feat/add-rememberme-checkbox branch November 13, 2025 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants