Skip to content

Fix sorting admin user list by last login (#22081) #22107

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Dec 13, 2022
Merged

Fix sorting admin user list by last login (#22081) #22107

merged 3 commits into from
Dec 13, 2022

Conversation

aceArt-GmbH
Copy link
Contributor

Backport of #22081

@aceArt-GmbH
Fix sorting admin user list by last login
0df7293 
Admin User Account list sort

`updated_unix` is not the same as "Last Sign-In"
@aceArt-GmbH aceArt-GmbH changed the title Fix sorting admin user list by last login Backport: Fix sorting admin user list by last login Dec 12, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Dec 12, 2022
@lunny lunny added this to the 1.18.0 milestone Dec 12, 2022
@lunny lunny added type/bug skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels Dec 12, 2022
@techknowlogick techknowlogick changed the title Backport: Fix sorting admin user list by last login Fix sorting admin user list by last login (#22081) Dec 12, 2022
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Dec 12, 2022
@jolheiser
Copy link
Member

🎺 🤖

@jolheiser jolheiser merged commit 1409b34 into go-gitea:release/v1.18 Dec 13, 2022
@aceArt-GmbH aceArt-GmbH deleted the fixAdminUserSort1.18 branch December 14, 2022 07:09
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lunny lunny lunny approved these changes

@silverwind silverwind silverwind approved these changes

@KN4CK3R KN4CK3R KN4CK3R approved these changes

Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Projects
None yet
Milestone
1.18.0
Development

Successfully merging this pull request may close these issues.
6 participants
@aceArt-GmbH @jolheiser @lunny @silverwind @KN4CK3R @GiteaBot