Skip to content

Commit

Permalink
community: fix "confluence-loader" enable include_labels for document…
Browse files Browse the repository at this point in the history
…s loaded via CQL (#29089)

## Description
This PR enables label inclusion for documents loaded via CQL in the
confluence-loader.

- Updated _lazy_load to pass the include_labels parameter instead of
False in process_pages calls for documents loaded via CQL.
- Ensured that labels can now be fetched and added to the metadata for
documents queried with cql.

## Related Modification History
This PR builds on the previous functionality introduced in
[#28259](#28259), which
added support for including labels with the include_labels option.
However, this functionality did not work as expected for CQL queries,
and this PR fixes that issue.

If the False handling was intentional due to another issue, please let
me know. I have verified with our Confluence instance that this change
allows labels to be correctly fetched for documents loaded via CQL.

## Issue
Fixes #29088


## Dependencies
No changes.

## Twitter Handle
[@zenoengine](https://x.com/zenoengine)
  • Loading branch information
zenoengine authored Jan 8, 2025
1 parent 9d290ab commit c8ca1cd
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -408,7 +408,7 @@ def _lazy_load(self, **kwargs: Any) -> Iterator[Document]:
include_restricted_content,
include_attachments,
include_comments,
False, # labels are not included in the search results
include_labels,
content_format,
ocr_languages,
keep_markdown_format,
Expand Down

0 comments on commit c8ca1cd

Please sign in to comment.