Skip to content

Conversation

@fuero
Copy link
Contributor

@fuero fuero commented Sep 6, 2025

Apparantly the python jira module changed its API and the parameter must evaluate to false now.

Copy link
Collaborator

@ryneeverett ryneeverett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems reasonable since the python jira library suggests "If maxResults evaluates to False, it will try to get all issues in batches."

However, this isn't a recent change and the python library itself seems to always evaluate maxResults's falsiness (and None is falsey) so I suspect if this caused a recent breakage it is coming from the jira service itself since the python library seems to pass this value to the jira API.

As far as the API docs go, the result of either value would seem to be ambiguous. If this gives anyone more trouble perhaps it would be better to switch to 5000 which would seem to be the jira service's actual maximum? 🤷‍♂️

@ryneeverett
Copy link
Collaborator

@srl295 Any thoughts on this? Seems like it could plausibly be related to your issue.

@srl295
Copy link
Contributor

srl295 commented Sep 17, 2025

@srl295 Any thoughts on this? Seems like it could plausibly be related to your issue.

yeah maybe related to #1124 you mean?

Possibly although in my case it failed completely with that call, so maybe something changed 2 weeks ago and then more recently.

If you look at the suggested client-code patch from the jira python library it includes a pagination loop:

nextPageToken=next_page_token,

I don't know if that's handled by that library automatically or not. but it's possible that bugwarrior will have to add a pagination loop.

Copy link
Contributor

@srl295 srl295 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nevertheless it seems like a reasonable change

@ryneeverett ryneeverett merged commit 07d1d7e into GothenburgBitFactory:develop Sep 17, 2025
9 checks passed
@srl295
Copy link
Contributor

srl295 commented Sep 17, 2025

OK ,i can confirm my local copy (other PR) is only getting 50 tickets… 

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.

3 participants