Skip to content

fix: The import document for the Web knowledge base displays an error - #6585

Merged
wangdan-fit2cloud merged 1 commit into
v2from
pr@v2@fit_document_import
Aug 5, 2026
Merged

fix: The import document for the Web knowledge base displays an error#6585
wangdan-fit2cloud merged 1 commit into
v2from
pr@v2@fit_document_import

Conversation

@shaohuzhang1

Copy link
Copy Markdown
Contributor

fix: The import document for the Web knowledge base displays an error

Copilot AI lite review requested due to automatic review settings August 5, 2026 02:26
@shaohuzhang1

Copy link
Copy Markdown
Contributor Author

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adjusts UI conditional logic related to “web document”/knowledge-base type handling to address an error shown during web knowledge-base document import.

Changes:

  • Updated template conditions to treat the “type” value as numeric rather than strictly the string '1'.
  • Adjusted the import dialog label logic for “allow preview/download” based on the (web) type.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ui/src/views/paragraph/index.vue Adjusts web-document detection in the paragraph header to control display of the source URL.
ui/src/views/document/component/ImportDocumentDialog.vue Adjusts label selection for the allow_download checkbox based on knowledge/document type.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

<back-button to="-1" style="margin-left: -4px"></back-button>
<h3 style="display: inline-block">{{ documentDetail?.name }}</h3>
<el-text type="info" v-if="documentDetail?.type === '1'"
<el-text type="info" v-if="documentDetail?.type == 1"
<el-checkbox v-model="form.allow_download">
{{
type === '1'
type == 1
@wangdan-fit2cloud
wangdan-fit2cloud merged commit 955e5a4 into v2 Aug 5, 2026
5 of 7 checks passed
@wangdan-fit2cloud
wangdan-fit2cloud deleted the pr@v2@fit_document_import branch August 5, 2026 02:33
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