Remove useSwipeReview from server

server/lib/models/user.js

@@ -7,7 +7,6 @@ module.exports = (sequelize, DataTypes) => {
     // given the structures of our queries we need to do. If somebody wants to do it, go for it!
     // For now, this takes a comma-separated list of language codes
     languages: DataTypes.STRING,
-    useSwipeReview: DataTypes.BOOLEAN,
   }, {});
 
   return User;

server/lib/users.js

@@ -13,20 +13,18 @@ module.exports = {
 
 async function get(email) {
   debug('GETTING_USER', email);
-  const [user] = await User.findAll({ where: { email }});
+  const [user] = await User.findAll({ where: { email } });
   const userLanguages = user.languages || '';
   return {
     email: user.email,
     languages: userLanguages.split(',').filter(Boolean),
-    settings: {
-      useSwipeReview: user.useSwipeReview || false,
-    },
+    settings: {},
   };
 }
 
 function createUserIfNecessary(email) {
   debug('CREATE_USER_IF_NECESSARY', email);
-  return User.findOrCreate({ where: { email }});
+  return User.findOrCreate({ where: { email } });
 }
 
 function updateSetting(email, key, value) {
@@ -49,7 +47,7 @@ function updateSetting(email, key, value) {
 
 async function addLanguage(email, language) {
   debug('ADDING_LANGUAGE', email, language);
-  const [user] = await User.findAll({ where: { email }});
+  const [user] = await User.findAll({ where: { email } });
   if (!user) {
     return;
   }
@@ -75,7 +73,7 @@ async function addLanguage(email, language) {
 
 async function removeLanguage(email, language) {
   debug('REMOVING_LANGUAGE', email, language);
-  const [user] = await User.findAll({ where: { email }});
+  const [user] = await User.findAll({ where: { email } });
   if (!user) {
     return;
   }

server/migrations/20210724095500-remove-useSwipeReview.js

@@ -0,0 +1,22 @@
+'use strict';
+
+module.exports = {
+  up: (queryInterface) => {
+    return queryInterface.removeColumn(
+      'Users',
+      'useSwipeReview',
+    );
+
+  },
+
+  down: (queryInterface, Sequelize) => {
+    return queryInterface.addColumn(
+      'Users',
+      'useSwipeReview',
+      {
+        type: Sequelize.BOOLEAN,
+        defaultValue: false,
+      },
+    );
+  },
+};

server/tests/lib/users.test.js

@@ -6,7 +6,6 @@ import users from '../../lib/users';
 const exampleUserRecord = {
   id: '1',
   email: 'foo@example.com',
-  useSwipeReview: true,
   languages: '',
 };
 
@@ -31,16 +30,14 @@ test.serial('get: should get user', async (t) => {
   t.deepEqual(user, {
     email: 'foo@example.com',
     languages: [],
-    settings: {
-      useSwipeReview: true,
-    },
+    settings: {},
   });
 });
 
 test.serial('updateSetting: should update user', async (t) => {
-  await users.updateSetting('foo@example.com', 'useSwipeReview', true);
+  await users.updateSetting('foo@example.com', 'foo', true);
   t.true(User.update.calledWith({
-    useSwipeReview: true,
+    foo: true,
   }, {
     where: {
       email: 'foo@example.com',