List a team's member API page parameter is 0-based (should be 1-based) #10854

Closed
opened 2025-11-02 09:20:04 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @yamadayutaka on GitHub (May 16, 2023).

Description

In the API specification, page is supposed to be specified as 1-based.

https://try.gitea.io/api/swagger#/organization/orgListTeamMembers

image

However, it seems to be implemented with 0-based. (Please see screenshots)

Gitea Version

1.19.3

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

Target Organization/Team

image

Result when page=1 is specified

image

Result when page=0 is specified

image

Git Version

No response

Operating System

No response

How are you running Gitea?

Installed with Helm chart and running on Kubernetes.

Override values

service:
  http:
    type: LoadBalancer

Install command

helm install gitea gitea-charts/gitea --version 8.3.0 -f override-values.yaml

Database

None

Originally created by @yamadayutaka on GitHub (May 16, 2023). ### Description In the API specification, `page` is supposed to be specified as `1-based`. https://try.gitea.io/api/swagger#/organization/orgListTeamMembers ![image](https://github.com/go-gitea/gitea/assets/77612853/a9b71199-fc5e-429c-9635-437efbfdf1d1) However, it seems to be implemented with `0-based`. (Please see screenshots) ### Gitea Version 1.19.3 ### Can you reproduce the bug on the Gitea demo site? No ### Log Gist _No response_ ### Screenshots ### Target Organization/Team ![image](https://github.com/go-gitea/gitea/assets/77612853/0c061646-cfb9-4579-8304-07623029587f) ### Result when `page=1` is specified ![image](https://github.com/go-gitea/gitea/assets/77612853/797a91bb-d97c-454c-a2ea-b2299e61ffe7) ### Result when `page=0` is specified ![image](https://github.com/go-gitea/gitea/assets/77612853/13e01c67-0a68-4413-8c06-d72fb14bfeeb) ### Git Version _No response_ ### Operating System _No response_ ### How are you running Gitea? Installed with Helm chart and running on Kubernetes. ### Override values ```yaml service: http: type: LoadBalancer ``` ### Install command ``` helm install gitea gitea-charts/gitea --version 8.3.0 -f override-values.yaml ``` ### Database None
GiteaMirror added the issue/confirmedtype/bug labels 2025-11-02 09:20:04 -06:00
Author
Owner

@yardenshoham commented on GitHub (May 16, 2023):

The fix would be a breaking change so this shouldn't go in 1.19.4

@yardenshoham commented on GitHub (May 16, 2023): The fix would be a breaking change so this shouldn't go in 1.19.4
Author
Owner

@lunny commented on GitHub (May 16, 2023):

It should be a bug, because the API documentation said it's 1 based.

@lunny commented on GitHub (May 16, 2023): It should be a bug, because the API documentation said it's 1 based.
Author
Owner

@yardenshoham commented on GitHub (May 16, 2023):

I agree it's a bug. The fix itself would be a breaking change so it should be fixed in 1.20

@yardenshoham commented on GitHub (May 16, 2023): I agree it's a bug. The fix itself would be a breaking change so it should be fixed in 1.20
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#10854