Skip to content
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

bug:流水线描述内容过长导致编辑后发布报错500 #11404 #11411

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@
hoverUnfold: {
type: Boolean,
default: false
},
maxlength: {
type: String,
default: ''
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里不要用字符串,明明是一个数字,转成number

Copy link
Collaborator

Choose a reason for hiding this comment

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

默认值,怎么还是字符串

}
},
methods: {
Expand All @@ -28,7 +32,7 @@
}
},
render (h) {
const { value, readOnly, handleInput, name, handleBlur, title, clickUnfold, hoverUnfold, placeholder, disabled } = this
const { value, readOnly, handleInput, name, handleBlur, title, clickUnfold, hoverUnfold, placeholder, disabled, maxlength } = this
return (
<textarea
placeholder={placeholder}
Expand All @@ -39,6 +43,7 @@
name={name}
disabled={readOnly || disabled}
value={value}
maxlength={maxlength}
/>
)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@
:disabled="!editable"
name="desc"
:value="pipelineSetting.desc"
maxlength="100"
:placeholder="$t('pipelineDescInputTips')"
v-validate.initial="'max:100'"
:handle-change="handleBaseInfoChange"
Expand Down