Skip to content

Conversation

@Mai-Lapyst
Copy link
Contributor

@Mai-Lapyst Mai-Lapyst commented May 24, 2022

Fixes #19791 by adding an check if filtering after any repo; if yes, simply set the total count for the pageing to the sum of the issue count for each selected repo by utilize issueCountByRepo.

Fix #19791

@codecov-commenter
Copy link

codecov-commenter commented May 25, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@12c742f). Click here to learn what that means.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main   #19801   +/-   ##
=======================================
  Coverage        ?   47.27%           
=======================================
  Files           ?      958           
  Lines           ?   133506           
  Branches        ?        0           
=======================================
  Hits            ?    63114           
  Misses          ?    62725           
  Partials        ?     7667           
Impacted Files Coverage Δ
routers/web/user/home.go 65.49% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 12c742f...403406d. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 25, 2022
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels May 25, 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 May 25, 2022
@6543 6543 added this to the 1.17.0 milestone May 25, 2022
@6543
Copy link
Member

6543 commented May 25, 2022

please update branch or allow maintainers to edit

@Mai-Lapyst Mai-Lapyst force-pushed the fix-wrong-issue-counting-for-pageing-on-issue-dashboard branch from bd17cab to 223afc4 Compare May 26, 2022 19:22
@Mai-Lapyst
Copy link
Contributor Author

please update branch or allow maintainers to edit

Have rebased; you also should now be able to edit the branch if further rebasing is necessary

@zeripath zeripath changed the title Fixing wrong pageing when filtering on the issue dashboard Fixing wrong paging when filtering on the issue dashboard Jun 4, 2022
@zeripath zeripath merged commit 090d89e into go-gitea:main Jun 4, 2022
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jun 5, 2022
* giteaofficial/main:
  Add alt text to logo (go-gitea#19892)
  Limit max-height of CodeMirror editors for issue comment and wiki (go-gitea#18271)
  Implement http signatures support for the API (go-gitea#17565)
  Increment tests time out from 40m to 50m because sometimes the machine is slow (go-gitea#19887)
  fix(CI/CD): correct CI variable. (go-gitea#19886)
  Fix typo (go-gitea#19889)
  Fixing wrong paging when filtering on the issue dashboard (go-gitea#19801)
  Move `/info` outside authorization (go-gitea#19888)
  Fix order by parameter (go-gitea#19849)
  Exclude Archived repos from Dashboard Milestones (go-gitea#19882)
  use exact search instead of fuzzy search for branch filter dropdown (go-gitea#19885)
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this pull request Aug 24, 2022
…9801)

Fixes go-gitea#19791 by adding an check if filtering after any repo; if yes, simply set the total count for the pageing to the sum of the issue count for each selected repo by utilize `issueCountByRepo`.

Fix go-gitea#19791
@Mai-Lapyst Mai-Lapyst deleted the fix-wrong-issue-counting-for-pageing-on-issue-dashboard branch September 2, 2022 12:02
@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.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Paging of User Issue List across repos does not change pagecount when applying filters

8 participants