search

MrRefactoring / jira.js

A JavaScript/TypeScript wrapper for the JIRA Cloud, Service Desk and Agile REST API
https://mrrefactoring.github.io/jira.js/
MIT License
349 stars 46 forks source link

bulkGetUser: accountId parameter doesn't get serialised #219

Closed iovecoldpizza closed 1 year ago

iovecoldpizza commented 2 years ago

Problem: The accountId param for bulkGetUser should be serialised, just like bulkGetUsersMigration, in order for it to return the correct results. The current version of bulkGetUser can only fetch one user at a time.

Current behaviour & code:

Screenshot 2022-07-02 at 15 34 52 Screenshot 2022-07-02 at 15 31 32

Expected behaviour & code:

Screenshot 2022-07-02 at 15 35 11 Screenshot 2022-07-02 at 15 29 08

Thanks!

MrRefactoring commented 1 year ago

Sorry for the late fix, had originally planned more changes, but didn't calculate the time commitment.

Fixed in v2.15.3

iovecoldpizza commented 1 year ago

No prob man. Thanks for the effort. Would you prefer I create a PR for known fixes like this next time?

MrRefactoring commented 1 year ago

If you could do this, then it would be fine, it would speed up the fixing of problems.