From 347b7a639bdb8a46febc58694b66fb0fe83c1f6c Mon Sep 17 00:00:00 2001
From: ansuz <ansuz@transitiontech.ca>
Date: Thu, 22 Aug 2019 14:40:01 +0200
Subject: [PATCH] update XXX comment

---
 historyKeeper.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/historyKeeper.js b/historyKeeper.js
index 4b4e0ba58..6ea0a2324 100644
--- a/historyKeeper.js
+++ b/historyKeeper.js
@@ -853,8 +853,8 @@ module.exports.create = function (cfg) {
             // parsed[3] is the last known hash (optionnal)
             sendMsg(ctx, user, [seq, 'ACK']);
 
-            // XXX should we send metadata here too?
-            // my gut says yes
+            // FIXME should we send metadata here too?
+            // none of the clientside code which uses this API needs metadata, but it won't hurt to send it (2019-08-22)
             getHistoryAsync(ctx, parsed[1], -1, false, (msg, readMore) => {
                 if (!msg) { return; }
                 sendMsg(ctx, user, [0, HISTORY_KEEPER_ID, 'MSG', user.id, JSON.stringify(['FULL_HISTORY', msg])], readMore);