nodebb-plugin-web-push 0.5.1 → 0.6.1

package/library.js

@@ -10,6 +10,7 @@ const user = require.main.require('./src/user');
 const meta = require.main.require('./src/meta');
 const utils = require.main.require('./src/utils');
 const translator = require.main.require('./src/translator');
+const notifications = require.main.require('./src/notifications');
 
 const controllers = require('./lib/controllers');
 const subscriptions = require('./lib/subscriptions');
@@ -92,11 +93,15 @@ plugin.addRoutes = async ({ router, middleware, helpers }) => {
 			return helpers.notAllowed(req, res);
 		}
 
+		const { userLang } = await user.getSettings(req.uid);
 		const { subscription } = req.body;
-		await webPush.sendNotification(subscription, JSON.stringify({
-			title: 'Test notification',
-			body: 'This is a test message sent from NodeBB',
-		}));
+		const payload = await constructPayload({
+			nid: utils.generateUUID(),
+			bodyShort: 'Test notification',
+			bodyLong: 'This is a test message sent from NodeBB',
+			path: `${nconf.get('url')}/me/web-push`,
+		}, req.uid, userLang);
+		await webPush.sendNotification(subscription, JSON.stringify(payload));
 	});
 };
 
@@ -121,7 +126,7 @@ plugin.onNotificationPush = async ({ notification, uidsNotified: uids }) => {
 	db.pexpire(refKey, 1000 * 60 * 60 * 48); // only track last 48 hours
 
 	let payloads = await Promise.all(uids.map(async (uid, idx) => {
-		const payload = await constructPayload(notification, userSettings[idx].userLang);
+		const payload = await constructPayload(notification, uid, userSettings[idx].userLang);
 		return [uid, payload];
 	}));
 	payloads = new Map(payloads);
@@ -180,15 +185,28 @@ plugin.addProfileItem = async (data) => {
 	return data;
 };
 
-async function constructPayload({ nid, mergeId, bodyShort, bodyLong, path }, language) {
-	let { maxLength } = await meta.settings.get('web-push');
+async function constructPayload(notification, uid, lang) {
+	let { maxLength, icon, badge } = await meta.settings.get('web-push');
 	maxLength = parseInt(maxLength, 10) || 256;
 
-	if (!language) {
-		language = meta.config.defaultLang || 'en-GB';
+	// i18n/rtl
+	if (!lang) {
+		lang = meta.config.defaultLang || 'en-GB';
+	}
+	const dir = await translator.translate('[[language:dir]]', lang);
+
+	// Merge with related unread notifications
+	if (notification.mergeId) {
+		const related = await notifications.findRelated([notification.mergeId], `uid:${uid}:notifications:unread`);
+		const merged = await notifications.getMultiple(related).then(notifications.merge);
+		if (merged.length) {
+			notification = merged.pop();
+		}
 	}
 
-	let [title, body] = await translator.translateKeys([bodyShort, bodyLong], language);
+	const { nid, mergeId, bodyShort, bodyLong, path } = notification;
+
+	let [title, body] = await translator.translateKeys([bodyShort, bodyLong], lang);
 	([title, body] = [title, body].map(str => validator.unescape(utils.stripHTMLTags(str))));
 	const tag = mergeId || nid;
 	const url = `${nconf.get('url')}${path}`;
@@ -204,11 +222,16 @@ async function constructPayload({ nid, mergeId, bodyShort, bodyLong, path }, lan
 		body = `${body.slice(0, maxLength)}…`;
 	}
 
+	icon = icon || `${nconf.get('url')}/apple-touch-icon`;
+	badge = badge || `${nconf.get('url')}/apple-touch-icon`;
+
 	return {
 		title,
 		body,
 		tag,
-		data: { url },
+		lang,
+		dir,
+		data: { url, icon, badge },
 	};
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-web-push",
-  "version": "0.5.1",
+  "version": "0.6.1",
   "description": "A starter kit for quickly creating NodeBB plugins",
   "main": "library.js",
   "repository": {

package/templates/admin/plugins/web-push.tpl

@@ -15,6 +15,24 @@
 							Due to a software limitation, if the message body is greater than 4096 bytes, the message itself will be an attachment in the push notification.
 						</p>
 					</div>
+
+					<div class="mb-3">
+						<label class="form-label" for="badge">Badge URL</label>
+						<input type="text" id="badge" name="badge" title="Badge" class="form-control" placeholder="https://...">
+						<p class="form-text">
+							Optional — overrides the badge for messages sent (usually seen in the notification bar on mobile devices)
+							By default, the site's configured "touch icon" is sent.
+						</p>
+					</div>
+
+					<div class="mb-3">
+						<label class="form-label" for="icon">Icon URL</label>
+						<input type="text" id="icon" name="icon" title="Icon" class="form-control" placeholder="https://...">
+						<p class="form-text">
+							Optional — overrides the icon for messages sent (can be used for branding, etc.)
+							By default, the site's configured "touch icon" is sent.
+						</p>
+					</div>
 				</div>
 			</form>