-
Notifications
You must be signed in to change notification settings - Fork 664
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
[Jira] Fix the parameter of remove_user_from_group from username to account id #1018
[Jira] Fix the parameter of remove_user_from_group from username to account id #1018
Conversation
@YugoHino Can we support for the DC release as well ? :) |
@gonchik Thank you for the comment. Your comment is correct, I only cared about Cloud version but I should care about DC as well. I will change the approach that currently replace username to account_id but username should be there and account_id should be added as optional. I will fix it. |
Hi, I fixed it. Those params are selected if the url domain is Cloud or not. When the url domain has atlassian.net, accountId is used for params. And else, username is used for it as well as so far. |
add docs
…YugoHino/atlassian-python-api into fix-jira-remove_user_from_group
Hi @gonchik Sorry for bothring you. Regarding this PR, please let me know if I should do for it. Or, I wait for the review. |
Hi @YugoHino , |
Codecov Report
@@ Coverage Diff @@
## master #1018 +/- ##
==========================================
- Coverage 36.00% 35.98% -0.02%
==========================================
Files 35 35
Lines 6777 6780 +3
Branches 1056 1057 +1
==========================================
Hits 2440 2440
- Misses 4231 4234 +3
Partials 106 106
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
…ccount id (#1018) * Fix the parameter from username to account id * fix remove_user_from_group for both DC and Cloud * fix optional parameter syntax * add docs * fix optional parameter syntax add docs * fix format by black
According to the API document, username is no longer available for remove_user_from_group. When this api is called, there is an error "HTTPError: Specified user does not exist or you do not have required permissions".
This PR is fixed the parmeter as following,
username -> accountId
Please let me know if I need to fix something.