Skip to content

chore(Transfer, TypeScript): fix event list response types#754

Merged
jbottigliero merged 1 commit intoglobus:mainfrom
theroncross:event-list-paging
Feb 17, 2026
Merged

chore(Transfer, TypeScript): fix event list response types#754
jbottigliero merged 1 commit intoglobus:mainfrom
theroncross:event-list-paging

Conversation

@theroncross
Copy link
Contributor

@theroncross theroncross commented Feb 13, 2026

The Transfer service returns a total property from both the user and endpoint-manager /event_list endpoints. This value is not included in some other responses that use offset paging.

Similarly, the event_list responses do not include has_next_page, so I've marked it as optional.

Finally, the DATA_TYPE property from this endpoint was not quite correct.

The transfer service returns a `total` property from both the user and
endpoint-manager `/event_list` endpoints. This value is *not* included in
some other responses that use [offset paging](https://docs.globus.org/api/transfer/overview/#offset_paging)

Similaly, this `event_list` response does *not* include `has_next_page`,
so I've marked it as optional.

Finally, the `DATA_TYPE` property from this endpoint was not quite
correct.
@jbottigliero
Copy link
Member

Just want to clarify, from Transfer's end total is related to the offset paging specifically and doesn't happen to just be included on these resources?

@theroncross
Copy link
Contributor Author

theroncross commented Feb 16, 2026

total is included in /task_list responses, but not /endpoint_search responses, both of which use offset paging.

edit: Apparently the total is expensive to calculate and will not be included in this response in the v2 api. I let the team know that it was useful to us, so we may end up with another way to access the total.

@jbottigliero jbottigliero merged commit f7826bc into globus:main Feb 17, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants