Updating backend for public go links (performed cleanup as well as setup

a new migration for the table information)

Author: Scott

db/migrations/20150809163538-links.js

@@ -9,14 +9,6 @@ export function up(queryInterface, Sequelize) {
       type: Sequelize.STRING,
       allowNull: false,
     },
-    goDescription: {
-      type: Sequelize.STRING,
-      allowNull: false,
-    },
-    publicGO: {
-      type: Sequelize.STRING,
-      allowNull: false,
-    },
     createdAt: Sequelize.DATE,
     updatedAt: Sequelize.DATE,
   });

db/migrations/20190223151710-add-columns-to-links.js

@@ -0,0 +1,16 @@
+export function up(queryInterface, Sequelize) {
+  return queryInterface.addColumn('links', 'description', {
+    type: Sequelize.STRING,
+    allowNull: true,
+  })
+  .then(() => queryInterface.addColumn('links', 'public', {
+    type: Sequelize.BOOLEAN,
+    allowNull: false,
+    defaultValue: false,
+  }));
+}
+
+export function down(queryInterface) {
+  return queryInterface.removeColumn('links', 'description')
+  .then(() => queryInterface.removeColumn('links', 'public'));
+}

models/link.js

@@ -21,29 +21,24 @@ export default sequelize.define('links', {
       },
     },
   },
-  goDescription: {
+  description: {
     type: DataTypes.STRING,
-    allowNull: false,
-    validate: {
-      notEmpty: true
-    }
+    allowNull: true,
   },
-  publicGO: {
-    type: DataTypes.STRING,
+  public: {
+    type: DataTypes.BOOLEAN,
     allowNull: false,
-    validate: {
-      notEmpty: true
-    },
   },
 }, {
   scopes: {
+    onlyPublic() {
+      return { where: { public: true } };
+    },
     paginate,
     orderBy(field, direction) {
       return sorting(field, direction, [
         'shortLink',
         'longLink',
-        'goDescription',
-        'publicGO',
         'createdAt',
         'updatedAt',
       ]);

package.json

@@ -58,7 +58,7 @@
     "pg": "^4.4.1",
     "pg-hstore": "^2.3.2",
     "sequelize": "^3.19.0",
-    "sqlite3": "^3.0.10",
+    "sqlite3": "^3.1.13",
     "umzug": "^1.6.0"
   },
   "devDependencies": {

routes/links.js

@@ -10,7 +10,7 @@ const router = Router(); // eslint-disable-line new-cap
 router
   .route('/')
     .get(paginate, sorting, (req, res, next) => {
-      const scopes = scopify(req.query);
+      const scopes = scopify(req.query, 'onlyPublic');
       Link.scope(scopes)
         .findAndCountAll({
           order: '"createdAt" DESC',
@@ -27,9 +27,9 @@ router
       Link.create({
         shortLink: req.body.shortLink.toLocaleLowerCase(),
         longLink: req.body.longLink,
-        goDescription: req.body.goDescription,
-        publicGO: req.body.publicGO,
-      }, { fields: ['shortLink', 'longLink', 'goDescription', 'publicGO'] })
+        description: req.body.description,
+        public: req.body.public,
+      }, { fields: ['shortLink', 'longLink', 'description', 'public'] })
         .then(link => res.status(201).send(link))
         .catch((err) => {
           err.status = 422;
@@ -73,7 +73,7 @@ router
         .then((link) => {
           if (link) {
             return link.updateAttributes(req.body, {
-              fields: ['shortLink', 'longLink', 'goDescription', 'publicGO'],
+              fields: ['shortLink', 'longLink', 'description', 'public'],
             });
           }
           return Promise.reject({ message: 'Link not found', status: 404 });