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

Fix problem when database id is not increment as expected #17124

Merged
merged 1 commit into from
Sep 22, 2021

Conversation

lunny
Copy link
Member

@lunny lunny commented Sep 22, 2021

Even if the id is not sequence increment, this PR will also make actions list accroding created time.

@lunny lunny added the type/enhancement An improvement of existing functionality label Sep 22, 2021
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 22, 2021
@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 Sep 22, 2021
@lafriks lafriks merged commit d9e237e into go-gitea:main Sep 22, 2021
@lunny lunny deleted the lunny/feed_created branch September 23, 2021 04:28
finga pushed a commit to finga/gitea that referenced this pull request Oct 6, 2021
Although go-gitea#17124 fixed the same issue for the feed, some other parts
with the same issue were found.
jolheiser pushed a commit that referenced this pull request Oct 6, 2021
Although #17124 fixed the same issue for the feed, some other parts
with the same issue were found.

Co-authored-by: finga <finga@onders.org>
Chianina pushed a commit to Chianina/gitea that referenced this pull request Oct 15, 2021
…#17229)

Although go-gitea#17124 fixed the same issue for the feed, some other parts
with the same issue were found.

Co-authored-by: finga <finga@onders.org>
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
@wxiaoguang wxiaoguang added this to the 1.16.0 milestone Nov 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants