Skip to content

fix: update get_user_list method to include user_id for workspace validation - #6589

Closed
shaohuzhang1 wants to merge 1 commit into
v2from
pr@v2@fit_document_import
Closed

fix: update get_user_list method to include user_id for workspace validation#6589
shaohuzhang1 wants to merge 1 commit into
v2from
pr@v2@fit_document_import

Conversation

@shaohuzhang1

Copy link
Copy Markdown
Contributor

fix: update get_user_list method to include user_id for workspace validation

Copilot AI lite review requested due to automatic review settings August 5, 2026 07:02
@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 the document import/settings dialog in the Vue admin UI by conditionally hiding the “allow download/preview” option when the dialog is in import mode (isImport). Note: the PR title/description currently reference a backend get_user_list change, which does not match the diff shown.

Changes:

  • Hide the allow_download form item during import (v-if="!isImport").

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

</div>
</el-form-item>
<el-form-item prop="allow_download">
<el-form-item prop="allow_download" v-if="!isImport">
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