Skip to content

[Confluence] page_exists method without error log #949

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

Merged
merged 1 commit into from
Mar 3, 2022

Conversation

Starkandco
Copy link
Contributor

Confluence.py change to page_exists() function. Function will now make its own request and handle the response to determine the True or False value rather than utilising "self.get_page_by_title(space, title)" which outputs errors to the log if it cannot find a page.

page_exists() updated to make its own request and handle the response to return a True or False rather than utilising "self.get_page_by_title(space, title)" which outputs messages to the log if it cannot find a page.
@Starkandco
Copy link
Contributor Author

Relates to issue 547

@codecov-commenter
Copy link

Codecov Report

Merging #949 (358b258) into master (56d1d1f) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #949      +/-   ##
==========================================
- Coverage   36.50%   36.47%   -0.04%     
==========================================
  Files          34       34              
  Lines        6527     6533       +6     
  Branches     1007     1010       +3     
==========================================
  Hits         2383     2383              
- Misses       4037     4043       +6     
  Partials      107      107              
Impacted Files Coverage Δ
atlassian/confluence.py 12.75% <0.00%> (-0.07%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 56d1d1f...358b258. Read the comment docs.

@gonchik gonchik merged commit fc18646 into atlassian-api:master Mar 3, 2022
@Starkandco Starkandco deleted the Starkandco-patch-1 branch March 3, 2022 21:42
gonchik pushed a commit that referenced this pull request Sep 3, 2022
page_exists() updated to make its own request and handle the response to return a True or False rather than utilising "self.get_page_by_title(space, title)" which outputs messages to the log if it cannot find a page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants