Fetch all items where needed. #475

Merged
6543 merged 6 commits from noerw/tea:fix-464 into master 3 months ago
noerw commented 4 months ago
Collaborator

Disable pagination in all places where we need all items.
Ideally we'd do multiple paginated requests until the needed items are local, but this is blocked by gitea/go-sdk#473. So this is a stopgap to get correct behaviour.

Fixes #464

Disable pagination in all places where we need all items. Ideally we'd do multiple paginated requests until the needed items are local, but this is blocked by https://gitea.com/gitea/go-sdk/pulls/473. So this is a stopgap to get correct behaviour. Fixes #464
noerw added this to the v0.9.0 milestone 4 months ago
noerw added the
kind/bug
label 4 months ago
noerw added 1 commit 4 months ago
5b1999b794 explicitly disable pagination
lunny approved these changes 4 months ago
noerw added 1 commit 4 months ago
6543 added 1 commit 3 months ago
af48545453 Merge branch 'master' into fix-464
6543 approved these changes 3 months ago
Owner

Please resolve the conflicts.

Please resolve the conflicts.
noerw added 1 commit 3 months ago
b9a4e876d1 Merge branch 'master' into fix-464
Poster
Collaborator

@lunny there are no conflicts.
Looks like conflict detection is broken again/still. #427 has the same issue. I think in both cases I did a local merge, and after pushing that Gitea started to show conflicts.

@lunny there are no conflicts. Looks like conflict detection is broken again/still. #427 has the same issue. I think in both cases I did a local merge, and after pushing that Gitea started to show conflicts.
6543 closed this pull request 3 months ago
6543 reopened this pull request 3 months ago
6543 added 1 commit 3 months ago
e8758c330f
CI.restart()
6543 added 1 commit 3 months ago
15eafc7769 Merge branch 'master' into fix-464
6543 merged commit 883a27b14e into master 3 months ago
6543 referenced this issue from a commit 3 months ago
6543 deleted branch fix-464 3 months ago

Reviewers

lunny approved these changes 4 months ago
6543 approved these changes 3 months ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 883a27b14e.
Sign in to join this conversation.
Loading…
There is no content yet.