From beffb061a25031c3973147f31da7a4645f726351 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Jaenisch?= Date: Mon, 6 Jun 2022 00:20:38 +0200 Subject: [PATCH] A minimal change to replace data calls with attr as per guidelines MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This affects the manage topics on a repository. Namely the done button once changes are made. I'm unhappy with the associated template, as there is an anchor element for something that should be a button. Since it must not be mixed, I haven't refactored the code into a Vue component yet. Signed-off-by: André Jaenisch --- web_src/js/features/repo-home.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web_src/js/features/repo-home.js b/web_src/js/features/repo-home.js index f0ea18dd10c8a..c4bdf3d4506dc 100644 --- a/web_src/js/features/repo-home.js +++ b/web_src/js/features/repo-home.js @@ -30,7 +30,7 @@ export function initRepoTopicBar() { saveBtn.on('click', () => { const topics = $('input[name=topics]').val(); - $.post(saveBtn.data('link'), { + $.post(saveBtn.attr('data-link'), { _csrf: csrfToken, topics }, (_data, _textStatus, xhr) => {