Skip to content

[Bitbucket Cloud] Add support for query parameters in pullrequest.comments endpoint #1101

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

markarce
Copy link
Contributor

@markarce markarce commented Dec 16, 2022

Closes this issue.

Updated pull_requests.comments to accept optional query string and sort parameters, which are added to params if present and used with _get_paged.
Logic was copied from PullRequests.each, which already accepts query string and sort parameters.

…mments endpoint

Updated pull_requests.comments to accept a query string, which is passed to params if present and used with _get_paged
Updated to include the sort parameter
Reorganized docstring
@markarce markarce force-pushed the add_query_parameter_support_to_pullrequests_comments_endpoint branch from c79e695 to 249e630 Compare December 16, 2022 23:01
@codecov-commenter
Copy link

Codecov Report

Base: 36.05% // Head: 36.03% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (249e630) compared to base (6e7399b).
Patch coverage: 42.85% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1101      +/-   ##
==========================================
- Coverage   36.05%   36.03%   -0.02%     
==========================================
  Files          39       39              
  Lines        7101     7106       +5     
  Branches     1022     1024       +2     
==========================================
+ Hits         2560     2561       +1     
- Misses       4433     4435       +2     
- Partials      108      110       +2     
Impacted Files Coverage Δ
...ssian/bitbucket/cloud/repositories/pullRequests.py 87.19% <42.85%> (-1.70%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@gonchik gonchik merged commit 6965cc0 into atlassian-api:master Dec 19, 2022
@markarce markarce deleted the add_query_parameter_support_to_pullrequests_comments_endpoint branch December 19, 2022 18:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BitBucket Cloud] PullRequests.comments property does not properly handle query parameters
3 participants