-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
优化:上线单名称/工单名称超过50个字符时增加提示。 #2426
Conversation
pengkang1991
commented
Nov 30, 2023
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2426 +/- ##
=======================================
Coverage 76.54% 76.54%
=======================================
Files 115 115
Lines 15874 15874
=======================================
Hits 12150 12150
Misses 3724 3724 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Archery/sql/templates/sqlsubmit.html
Lines 331 to 348 in a564fe3
function validateForm(element) { | |
var result = true; | |
element.find('[required]').each( | |
function () { | |
var fieldElement = $(this); | |
//如果为null则设置为'' | |
var value = fieldElement.val() || ''; | |
if (value) { | |
value = value.trim(); | |
} | |
if (!value || value === fieldElement.attr('data-placeholder')) { | |
alert(fieldElement.attr('data-placeholder')); | |
result = false; | |
return result; | |
} | |
} | |
); | |
return result; |
在这实现吧
* 优化:上线单名称/工单名称超过50个字符时增加提示。 * 优化:上线单名称/工单名称超过50个字符时增加提示。 --------- Co-authored-by: kang.peng <kang.peng@qx.com>
* 优化:上线单名称/工单名称超过50个字符时增加提示。 * 优化:上线单名称/工单名称超过50个字符时增加提示。 --------- Co-authored-by: kang.peng <kang.peng@qx.com>