Bug(accommodations): add slugs to accommodations

- Adds a slug upon accommodation creation - Formats dates to a more readable format [Finishes#169385153]
andela · Oct 28, 2019 · 5928870 · 5928870
1 parent 076abd4
commit 5928870
Show file tree

Hide file tree

Showing 13 changed files with 28 additions and 18 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -83,6 +83,7 @@
     "request": "^2.87.0",
     "sequelize": "^5.19.1",
     "sequelize-cli": "^5.5.1",
+    "slugify": "^1.3.5",
     "swagger-jsdoc": "^3.4.0",
     "swagger-ui-express": "^4.1.1",
     "underscore": "^1.9.1"

diff --git a/src/controllers/accommodationController.js b/src/controllers/accommodationController.js
@@ -1,4 +1,5 @@
 import cloudinary from 'cloudinary';
+import slugify from 'slugify';
 import models from '../database/models';
 import strings from '../utils/stringsUtil';
 import responseUtil from '../utils/responseUtil';
@@ -29,7 +30,8 @@ export default class AccommodationController {
       highlights,
       amenities,
       owner: req.user.payload.id,
-      images
+      images,
+      slug: slugify(name.toLowerCase())
     };
     try {
       const newAccommodation = await models.accommodations.create(accommodation);

diff --git a/src/database/migrations/20190930143254-User.js b/src/database/migrations/20190930143254-User.js
@@ -27,7 +27,7 @@ module.exports = {
       allowNull: true
     },
     dob: {
-      type: Sequelize.DATE,
+      type: Sequelize.DATEONLY,
       allowNull: true
     },
     country: {
@@ -69,8 +69,8 @@ module.exports = {
         }
       }
     },
-    createdAt: Sequelize.DATE,
-    updatedAt: Sequelize.DATE,
+    createdAt: Sequelize.DATEONLY,
+    updatedAt: Sequelize.DATEONLY,
   }),
   down: queryInterface => queryInterface.dropTable('users')
 };
diff --git a/src/database/migrations/20191015081823-create-trip-types.js b/src/database/migrations/20191015081823-create-trip-types.js
@@ -13,11 +13,11 @@ module.exports = {
       },
       createdAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       },
       updatedAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       }
     });
   },

diff --git a/src/database/migrations/20191015081900-create-request-status.js b/src/database/migrations/20191015081900-create-request-status.js
@@ -13,11 +13,11 @@ module.exports = {
       },
       createdAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       },
       updatedAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       }
     });
   },

diff --git a/src/database/migrations/20191015082308-create-destinations.js b/src/database/migrations/20191015082308-create-destinations.js
@@ -31,11 +31,11 @@ module.exports = {
       },
       createdAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       },
       updatedAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       }
     });
   },

diff --git a/src/database/migrations/20191015091939-create-locations.js b/src/database/migrations/20191015091939-create-locations.js
@@ -17,11 +17,11 @@ module.exports = {
       },
       createdAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       },
       updatedAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       }
     });
   },

diff --git a/src/database/migrations/20191016090432-create-role.js b/src/database/migrations/20191016090432-create-role.js
@@ -16,11 +16,11 @@ module.exports = {
       },
       createdAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       },
       updatedAt: {
         allowNull: false,
-        type: Sequelize.DATE
+        type: Sequelize.DATEONLY
       }
     });
   },

diff --git a/src/database/migrations/20191017165632-create-accommodations.js b/src/database/migrations/20191017165632-create-accommodations.js
@@ -23,6 +23,7 @@ module.exports = {
       amenities: { type: Sequelize.STRING},
       owner: {type: Sequelize.INTEGER},
       images: {type: Sequelize.JSONB},
+      slug: {type: Sequelize.STRING},
       createdAt: {allowNull: false,type: Sequelize.DATEONLY},
       updatedAt: {allowNull: false,type: Sequelize.DATEONLY}
     });

diff --git a/src/database/models/accommodations.js b/src/database/models/accommodations.js
@@ -9,7 +9,8 @@ module.exports = (sequelize, DataTypes) => {
     highlights: DataTypes.STRING,
     amenities: DataTypes.STRING,
     owner: DataTypes.INTEGER,
-    images: DataTypes.JSONB
+    images: DataTypes.JSONB,
+    slug: DataTypes.STRING,
   }, { tableName: 'accommodations' });
   accommodations.associate = models => {
     // associations can be defined here

diff --git a/src/database/models/destinations.js b/src/database/models/destinations.js
@@ -3,8 +3,8 @@ module.exports = (sequelize, DataTypes) => {
   const destinations = sequelize.define('destinations', {
     locationId: DataTypes.INTEGER,
     bookingId: DataTypes.INTEGER,
-    arrivalDate: DataTypes.DATE,
-    departureDate: DataTypes.DATE,
+    arrivalDate: DataTypes.DATEONLY,
+    departureDate: DataTypes.DATEONLY,
     requestId: DataTypes.INTEGER,
     reasons: DataTypes.STRING,
     isFinal: DataTypes.BOOLEAN

diff --git a/src/database/models/users.js b/src/database/models/users.js
@@ -8,7 +8,7 @@ module.exports = (sequelize, Datatypes) => {
     role: Datatypes.INTEGER,
     phone: Datatypes.STRING,
     gender: Datatypes.STRING,
-    dob: Datatypes.DATE,
+    dob: Datatypes.DATEONLY,
     country: Datatypes.STRING,
     language: Datatypes.STRING,
     currency: Datatypes.STRING,