fix: machine unpair

feat: add removed_devices table

lib/pairing.js

@@ -4,6 +4,7 @@ const readFile = pify(fs.readFile)
 const db = require('./db')
 const options = require('./options')
 const logger = require('./logger')
+const uuid = require('uuid')
 
 // A machine on an older version (no multicassette code) could be paired with a server with multicassette code.
 // This makes sure that the server stores a default value
@@ -17,13 +18,24 @@ function pullToken (token) {
 }
 
 function unpair (deviceId) {
+  const backupSQL = `INSERT INTO
+  unpaired_devices(id, device_id, name, model, paired, unpaired)
+  VALUES (
+    $1,
+    $2,
+    (SELECT name FROM devices WHERE device_id=$2),
+    (SELECT model FROM devices WHERE device_id=$2),
+    (SELECT created FROM devices WHERE device_id=$2),
+    now()
+  )`
+
   // TODO new-admin: We should remove all configs related to that device. This can get tricky.
   return db.tx(t => {
     const q1 = t.none('DELETE FROM devices WHERE device_id=$1', [deviceId])
     const q2 = t.none('DELETE FROM machine_pings WHERE device_id=$1', [deviceId])
     const q3 = t.none('DELETE FROM machine_network_heartbeat WHERE device_id=$1', [deviceId])
     const q4 = t.none('DELETE FROM machine_network_performance WHERE device_id=$1', [deviceId])
-    return Promise.all([q1, q2, q3, q4])
+    return t.manyOrNone(backupSQL, [uuid.v4(), deviceId]).then(() => Promise.all([q1, q2, q3, q4]))
   })
 }
 

migrations/1637877732001-add_unpaired_devices_table.js

@@ -0,0 +1,22 @@
+var db = require('./db')
+
+exports.up = function (next) {
+  var sql = [
+    `ALTER TABLE cashbox_batches
+    DROP CONSTRAINT cashbox_batches_device_id_fkey;`,
+    `CREATE TABLE IF NOT EXISTS unpaired_devices (
+      id uuid PRIMARY KEY,
+      device_id text NOT NULL,
+      model text,
+      name text,
+      paired timestamp NOT NULL,
+      unpaired timestamp NOT NULL
+    )`,
+  ]
+
+  db.multi(sql, next)
+}
+
+exports.down = function (next) {
+  next()
+}