Skip to content

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