Skip to content

Commit

Permalink
Merge pull request #379 from silverstripe/pulls/4/update-js-1696119164
Browse files Browse the repository at this point in the history
DEP Update JS dependencies
  • Loading branch information
emteknetnz authored Oct 3, 2023
2 parents c21a6e0 + 4041209 commit ec8b363
Show file tree
Hide file tree
Showing 4 changed files with 2,215 additions and 1,936 deletions.
2 changes: 1 addition & 1 deletion client/dist/js/CommentsInterface.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion client/dist/js/jquery.min.js

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions client/src/legacy/CommentsInterface.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
* @package comments
*/
(function ($) {
// The above closure encapsulates the $ variable away from the global scope
// and the one below is the `$(document).ready(...)` shorthand.
// The above closure encapsulates the $ variable away from the global scope
// and the one below is the `$(document).ready(...)` shorthand.
$(() => {
// Override the default URL validator in order to extend it to allow protocol-less URLs
$.validator.methods.url = function (value, element) {
Expand Down Expand Up @@ -117,7 +117,7 @@
comment.html(html).addClass('spam');
}
},
failure(html) {
failure() {
const errorMsg = ss.i18n._t('CommentsInterface_singlecomment_ss.AJAX_ERROR');
alert(errorMsg);

Check warning on line 122 in client/src/legacy/CommentsInterface.js

View workflow job for this annotation

GitHub Actions / Update JS

Unexpected alert
}
Expand Down
Loading

0 comments on commit ec8b363

Please sign in to comment.