GoSimplicity commented on code in PR #8547:
URL:
https://github.com/apache/incubator-devlake/pull/8547#discussion_r2297084047
##########
backend/plugins/jira/tasks/epic_collector.go:
##########
@@ -98,6 +117,10 @@ func CollectEpics(taskCtx plugin.SubTaskContext)
errors.Error {
query.Set("startAt", fmt.Sprintf("%v",
reqData.Pager.Skip))
Review Comment:
Excellent catch on the pagination changes! You're absolutely correct that
api/3/search/jql uses nextPageToken instead of startAt. I'll update the
implementation to:
1.Use different pagination strategies based on the API version
2.Set concurrency to 1 for api/3 (sequential pagination only)
3.Handle the nextPageToken properly in the response parser
I'll revise the PR to address these issues. Would you prefer separate
collector implementations for each API version, or a single collector with
conditional logic?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
