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.

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

Fix UI pagingnum #20087

Closed
wants to merge 1 commit into from
Closed

Fix UI pagingnum #20087

wants to merge 1 commit into from

Conversation

lunny
Copy link
Member

@lunny lunny commented Jun 22, 2022

Fix #20080

@jolheiser jolheiser added this to the 1.16.9 milestone Jun 22, 2022
@jpraet
Copy link
Member

jpraet commented Jun 22, 2022

The bug report #20080 was about the activity feed on the dashboard, while this PR modifies the repo router?

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jun 22, 2022
@jpraet jpraet modified the milestones: 1.16.9, 1.17.0 Jun 22, 2022
@lunny lunny closed this Jun 23, 2022
@lunny lunny removed this from the 1.17.0 milestone Jun 23, 2022
@lunny
Copy link
Member Author

lunny commented Jun 23, 2022

replaced by #20094

@lunny lunny deleted the lunny/pagingnum branch June 23, 2022 05:29
@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
No reviews
Assignees
No one assigned
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Dashboard feed no longer respects FEED_PAGING_NUM setting
4 participants
@lunny @jpraet @GiteaBot @jolheiser