Skip to content

Changed group_members to be compatiable with newest Jira LTS #1944

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

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

Conversation

marcin-wozniak-wttech
Copy link

Hello!
We have tested that module in python with the newest JIRA LTS version 10.3.
In version 10.3 Atlassian has been decommissioned a endpoint[1] /rest/api/2/group with GET method.
They introduced a new endpoint /rest/api/2/group/member with GET method.

So I implemented it and tested on two version at 9.12 and 10.3 and it has got backword compatibility and the outputs are the same.

If I may help you please give me a shout!

--
Have a great day!
Marcin

--
[1] - https://confluence.atlassian.com/jirasoftware/jira-software-10-0-x-upgrade-notes-1431241649.html

@marcin-wozniak-wttech
Copy link
Author

Hello!
After quite a few attempts, I’ve finally pushed a new version! Apologies for all the noise in that PR - I hope you're not too annoyed with me 😄

Additionally, I’ve run:

pre-commit run --all-files -v

Would you prefer me to push the reformatted code as well, or just stick to the actual changes?

Thanks!

@adehad
Copy link
Contributor

adehad commented Apr 16, 2025

Could I ask you to rebase and resolve any conflicts, hard to see your specific changes as they seem to have been squashed with other commits on the history.

@studioj
Copy link
Collaborator

studioj commented Apr 27, 2025

Hello! After quite a few attempts, I’ve finally pushed a new version! Apologies for all the noise in that PR - I hope you're not too annoyed with me 😄

Additionally, I’ve run:

pre-commit run --all-files -v

Would you prefer me to push the reformatted code as well, or just stick to the actual changes?

Thanks!

keep to the actual changes please ... it will probably be clearer after that rebase which @adehad requested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants