Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use "agile" endpoint over now removed "greenhopper" #1186

Merged
merged 1 commit into from Oct 17, 2021

Conversation

adehad
Copy link
Collaborator

@adehad adehad commented Oct 16, 2021

The "greenhopper" endpoint is deprecated in Jira version ~6, our test instances don't support those endpoints any more and it is causing troubles to our users: #1146 and also with ranking #1160 .

Some details about deprecation can be found here:
https://community.atlassian.com/t5/Jira-Core-Server-questions/Greenhooper-REST-API/qaq-p/816504

https://web.archive.org/web/20131022032352/http://blogs.atlassian.com/2013/08/renaming-greenhopper-and-bonfire/

We also have this documented in our code:

jira/jira/resources.py

Lines 1187 to 1192 in fa5dea1

GREENHOPPER_REST_PATH = "greenhopper"
""" Old, private API. Deprecated and will be removed from Jira on the 1st February 2016. """
AGILE_EXPERIMENTAL_REST_PATH = "greenhopper/experimental-api"
""" Experimental API available in Jira Agile 6.7.3 - 6.7.6, basically the same as Public API """
AGILE_BASE_REST_PATH = "agile"
""" Public API introduced in Jira Agile 6.7.7. """

@adehad adehad added the bug label Oct 16, 2021
@adehad adehad requested a review from studioj October 16, 2021 16:13
@adehad
Copy link
Collaborator Author

adehad commented Oct 16, 2021

@studioj As it has been about 5 years since this endpoint's removal (renaming technically) and the package aims not to support older versions (especially ones we can't even test for) I think we should be happy with this change.

If users do want to use the older "greenhopper" version, they can still pass the endpoint manually as part of the options OR just use an older version of this package.

Copy link
Collaborator

@studioj studioj left a comment

Choose a reason for hiding this comment

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

in the assumption we do test agile stuff, good to go

@adehad adehad merged commit a900097 into main Oct 17, 2021
@adehad adehad deleted the feature/use_agile_not_greenhopper_as_default_agile_url branch October 17, 2021 17:05
svermeulen pushed a commit to svermeulen/jira that referenced this pull request Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants