Skip to content

fix: The default suffix for workflow file upload nodes is set to uppercase DOC and PPT #3124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

shaohuzhang1
Copy link
Contributor

fix: The default suffix for workflow file upload nodes is set to uppercase DOC and PPT

@@ -141,7 +141,7 @@
<p class="line-height-22 mt-4">
{{ $t('common.fileUpload.other') }}
<el-text class="color-secondary"
>{{
>{{
$t(
'views.applicationWorkflow.nodes.baseNode.FileUploadSetting.fileUploadType.otherText'
)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There seem to be two instances where there is an extra > character after $t() calls:

<el-text class="color-secondary">{{

Ensure that these should instead be:

<el-text class="color-secondary">{{ --}}

Additionally, it's a good practice to use single quotes around template literals to avoid syntax errors, especially when mixing inline and multi-line templates. Here’s how you can fix them:

Before:

'{{ $t('someKey', 'arg1 arg2') }}'

After replacing double/single quotes for consistency:

"$t('someKey', 'arg1 args2')"

Or using single quotes if prefered for clarity within the string itself:

'$t(\'someKey\', \'arg1 args2\')'

@shaohuzhang1 shaohuzhang1 deleted the pr@main@fix_file_upload branch May 21, 2025 05:24
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.

1 participant