Skip to content
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

Add Support for Filtering Confluence V2 Bulk #217

Merged
merged 2 commits into from
Jun 14, 2023

Conversation

squatched
Copy link
Contributor

@squatched squatched commented Jun 13, 2023

The Confluence V2 API Bulk get supports filtering by IDs and status. This adds
support for those and body-format in the form of an additional method BulkFiltered().

Fixes #218.

The Confluence V2 API Bulk get supports filtering by IDs. This adds
support for that parameter optionally and is completely backwards
compatible.
@codecov
Copy link

codecov bot commented Jun 13, 2023

Codecov Report

Merging #217 (a629f22) into main (cedda2e) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
+ Coverage   85.48%   85.53%   +0.04%     
==========================================
  Files         137      137              
  Lines       11626    11678      +52     
==========================================
+ Hits         9939     9989      +50     
- Misses        941      942       +1     
- Partials      746      747       +1     
Impacted Files Coverage Δ
confluence/internal/page_impl.go 90.97% <100.00%> (+0.97%) ⬆️

... and 2 files with indirect coverage changes

There are more parameters available to the bulk query which would,
unfortunately, not be backward compatible with current version of Bulk
so rather than affecting that, I'm switching to having one
implementation that can handle all the filtering and Bulk() calls it
without any filtering.
@squatched squatched changed the title Add Support for Filtering Confluence V2 Bulk By ID Add Support for Filtering Confluence V2 Bulk Jun 13, 2023
@ctreminiom
Copy link
Owner

Hi @squatched,

PR approved!!

Thanks for contributing on this project 👍🏽

@ctreminiom ctreminiom merged commit b6643b0 into ctreminiom:main Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Confluence
Projects
Status: Released
Development

Successfully merging this pull request may close these issues.

Allow Usage of Filtering Parameters to Conf V2 Pages Bulk Get API
2 participants