-
Notifications
You must be signed in to change notification settings - Fork 39
Description
Hello there!
I wanted to bring to your attention an issue that I've encountered while working with the SprintService and the Jira API version 3. It seems that despite the upgrade, the getSprintIssues method is still fetching data from an older API endpoint, resulting in mismatched response formats and causing type errors in the JSON mapper.
Issue Summary:
After upgrading jira API from version 2 to 3, the getSprintIssues
method in the SprintService continues to fetch data from the older API endpoint, which returns some fields in the old string format. However, the JSON mapper expects responses in the new adf format introduced in version 3, resulting in type errors.
Steps to Reproduce:
- Upgrade Jira API from version 2 to version 3.
- Call the
getSprintIssues
method from the SprintService. - Observe that the response is fetched from the 1.0 API endpoint, returning some fields string format.
- Attempt to parse the response using the JSON mapper, expecting the new adf format.
- Encounter type errors due to the mismatch between the expected and actual response formats.
Expected Behavior:
The SprintService should be updated to fetch data from the new API endpoint introduced in Jira API version 3, providing responses in the new adf format.
The SprintService should be able to handle the fetched data from the older API endpoint.
That older API endpoint does not support ADF.
The fields are returned as basic strings.
Proposed Solution:
Update the SprintService to fetch data from the new API endpoint introduced in Jira API version 3.
Error
[TypeError] JiraCloud\ADF\AtlassianDocumentFormat::__construct(): Argument #1 ($document) must be of type DH\Adf\Node\Block\Document|DH\Adf\Node\Node, string given, called in /vendor/netresearch/jsonmapper/src/JsonMapper.php on line 708
#0 /vendor/netresearch/jsonmapper/src/JsonMapper.php(706): JiraCloud\ADF\AtlassianDocumentFormat->__construct()
#1 /vendor/netresearch/jsonmapper/src/JsonMapper.php(324): JsonMapper->createInstance()
#2 /vendor/netresearch/jsonmapper/src/JsonMapper.php(327): JsonMapper->map()
#3 /vendor/netresearch/jsonmapper/src/JsonMapper.php(483): JsonMapper->map()
#4 /vendor/complex/jira-cloud-restapi/src/Sprint/SprintService.php(67): JsonMapper->mapArray()