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

fix getIssueChangelog() wrong endpoint #339

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zombieleet
Copy link

calling the getChangelog() endpoint with an issueNumber causes the below error

null for uri: https://jira.[company-name].com/rest/api/2/issue/ISSUE-KEY/changelog?startAt=0&maxResults=50

This PR fixes the issue by doing the below

  1. add expand property with value of changelog to the query object
  2. remove "/changelog" path as a suffix to pathname (/issue/${issueNumber/)

This issue was also present in the dotnet implementation as well, which was fixed using this approach
solidify/jira-azuredevops-migrator#13

1. add `expand` property with value of `changelog` to the query object
2. remove `"/changelog"` path as a suffix to `pathname` `(/issue/${issueNumber/)`
@pioug pioug self-requested a review March 8, 2022 15:04
@pioug
Copy link
Contributor

pioug commented Mar 8, 2022

They look like two different endpoints:

I am not familiar with Jira API, but I'd prefer to recommend getIssue(..., ..., "changelog") instead of turning getChangelog into an alias of getIssue

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.

None yet

2 participants