Skip to content

fix(knowledge):Fix KB document questions list check logic#2565

Merged
fangyinc merged 1 commit into
eosphoros-ai:mainfrom
vnicers:fix_doc_rename
Mar 31, 2025
Merged

fix(knowledge):Fix KB document questions list check logic#2565
fangyinc merged 1 commit into
eosphoros-ai:mainfrom
vnicers:fix_doc_rename

Conversation

@vnicers
Copy link
Copy Markdown
Contributor

@vnicers vnicers commented Mar 31, 2025

Description

When editing the name of a KB document, if there are no question and answer pairs by default, the frontend will not pass the questions field, and the questions on the backend are None instead of empty list

5d108cdaefd14c9700953814a923bdc

How Has This Been Tested?

dc9dce87a748de3c50f3c27b473b275

Snapshots:

image

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

@github-actions github-actions Bot added the fix Bug fixes label Mar 31, 2025
Copy link
Copy Markdown
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for your contribution.

Copy link
Copy Markdown
Member

@fangyinc fangyinc left a comment

Choose a reason for hiding this comment

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

LGTM

@fangyinc fangyinc merged commit 9719c0a into eosphoros-ai:main Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants