nodebb-plugin-discord-onekite 1.1.14 → 1.1.15

package/README.md

@@ -1,7 +1,8 @@
-# nodebb-plugin-discord-onekite v1.1.4.5
+# nodebb-plugin-discord-onekite v1.1.4.7
 
-Fixes ACP save (API route) + floating save button + toast.
-
-- Adds API route via `routeHelpers.setupApiRoute` so `api.post('/admin/...')` works.
-- Adds floating save button bottom-right.
-- Shows NodeBB toast on success.
+Discord output cleanup:
+- Removes the extra content line under the title.
+- Sends an embed where:
+  - title is clickable (embed.url)
+  - embed description contains the message excerpt
+- `content` is only used as a fallback if Discord rejects embeds.

package/library.js

@@ -129,8 +129,7 @@ async function getPostExcerpt(pid) {
 }
 
 async function buildPayload({ tid, pid, isReply }) {
-  let baseUrl = normalizeBaseUrl(meta.config.url || meta.config['url']);
-  if (!baseUrl) { baseUrl = 'https://www.onekite.com'; }
+  const baseUrl = normalizeBaseUrl(meta.config.url);
 
   const topicData = await topics.getTopicFields(tid, ['tid', 'uid', 'cid', 'title', 'slug', 'mainPid']);
   if (!topicData) return null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-discord-onekite",
-  "version": "1.1.14",
+  "version": "1.1.15",
   "description": "Discord webhook notifier for Onekite (NodeBB v4.x only)",
   "main": "library.js",
   "license": "MIT",

package/static/lib/admin.js

@@ -1,42 +1,22 @@
 'use strict';
 /* global $, app */
 
-define('admin/plugins/discord-onekite', ['api'], function (api) {
+define('admin/plugins/discord-onekite', ['settings'], function (settings) {
   const ACP = {};
 
-  function getMultiSelectValues(selector) {
-    const el = document.querySelector(selector);
-    if (!el) return [];
-    return Array.from(el.selectedOptions || []).map(o => o.value);
-  }
-
-  function doSave() {
-    const payload = {
-      webhookUrl: String($('#webhookUrl').val() || '').trim(),
-      notifyReplies: $('#notifyReplies').is(':checked'),
-      cids: getMultiSelectValues('#cids'),
-    };
-
-    // `api.post('/admin/...')` -> `/api/admin/...`
-    return api.post('/admin/plugins/discord-onekite/save', payload).then(function () {
-      if (window.app && typeof app.alertSuccess === 'function') {
-        app.alertSuccess('Paramètres enregistrés !');
-      }
-    }).catch(function (err) {
-      // eslint-disable-next-line no-console
-      console.error(err);
-      if (window.app && typeof app.alertError === 'function') {
-        app.alertError('Erreur lors de l’enregistrement');
-      }
-    });
-  }
-
   ACP.init = function () {
-    if (!$('.discord-onekite-settings').length) return;
+    const $form = $('.discord-onekite-settings');
+    if (!$form.length) return;
+
+    settings.sync('discord-onekite', $form);
 
-    $(document).off('click.discordOnekite', '#save').on('click.discordOnekite', '#save', function (e) {
+    $('#save').off('click.discordOnekite').on('click.discordOnekite', function (e) {
       e.preventDefault();
-      doSave();
+      settings.persist('discord-onekite', $form, function () {
+        if (window.app && typeof app.alertSuccess === 'function') {
+          app.alertSuccess('Paramètres enregistrés !');
+        }
+      });
     });
   };
 

package/templates/admin/plugins/discord-onekite.tpl

@@ -1,10 +1,6 @@
 <div class="acp-page-container">
-  <div class="d-flex justify-content-between align-items-start mb-3">
-    <div>
-      <h4 class="mb-1">Discord Onekite</h4>
-      <p class="text-muted mb-0">Notifications Discord via webhook.</p>
-    </div>
-  </div>
+  <h4>Discord Onekite</h4>
+  <p class="text-muted">Notifications Discord via webhook.</p>
 
   <form role="form" class="discord-onekite-settings">
     <div class="mb-3">
@@ -30,9 +26,7 @@
         Si aucune catégorie n’est sélectionnée : <strong>toutes les catégories</strong> seront notifiées.
       </p>
     </div>
-  </form>
 
-  <button id="save" class="btn btn-primary" style="position: fixed; right: 2rem; bottom: 2rem; z-index: 1030;">
-    <i class="fa fa-save"></i>
-  </button>
+    <button id="save" type="button" class="btn btn-primary">Enregistrer les paramètres</button>
+  </form>
 </div>