Admin should be able to upgrade user level #35
Conversation
- Models of tables re-constructed to take into account newly constructed tables - Models of User level , Notifications, Categories implemented
- Implemented route for admin notifications - Implemented route to change password - Implemented route to change user level - Implement route to change user categories - Default images are loaded from server side - New models for userlevel, notifications, category - admin logic created for admin routes - seed files for categories and levels implemented
- Refactored tests to implement Admin
- Implement suffiecient tests to give above average coverage of code. - Admin middleware for Admin routes - Seed logic creatred for app to ensure easier usability.
…153149549, #153149477, ]
| } | ||
| }); | ||
| }, | ||
| down: function(queryInterface, Sequelize) { |
There was a problem hiding this comment.
Expected method shorthand object-shorthand
Unexpected unnamed method 'down' func-names
Missing space before function parentheses space-before-function-paren
'Sequelize' is defined but never used no-unused-vars
| @@ -0,0 +1,36 @@ | |||
| 'use strict'; | |||
| module.exports = { | |||
| up: function(queryInterface, Sequelize) { | |||
There was a problem hiding this comment.
Expected method shorthand object-shorthand
Unexpected unnamed method 'up' func-names
Missing space before function parentheses space-before-function-paren
| @@ -0,0 +1,36 @@ | |||
| 'use strict'; | |||
There was a problem hiding this comment.
Expected newline after "use strict" directive lines-around-directive
'use strict' is unnecessary inside of modules strict
| } | ||
| }); | ||
| }).on('error', (err) => { | ||
| console.log(`Error: ${err.message}`); |
There was a problem hiding this comment.
Unexpected console statement no-console
| // optional logging... disable after it's working | ||
| console.log(`======WAKUP DYNO: HEROKU RESPONSE: ${chunk}`); | ||
| } catch (err) { | ||
| console.log(err.message); |
There was a problem hiding this comment.
Unexpected console statement no-console
| @@ -2,4 +2,6 @@ require('./user.spec'); | |||
| require('./books.spec'); | |||
| require('./userbooks.spec'); | |||
| require('./model.spec'); | |||
|
|
|||
| require('./categories.spec'); | |||
| require('./authenticate.spec'); | |||
There was a problem hiding this comment.
Unable to resolve path to module './authenticate.spec' import/no-unresolved
| @@ -2,4 +2,6 @@ require('./user.spec'); | |||
| require('./books.spec'); | |||
| require('./userbooks.spec'); | |||
| require('./model.spec'); | |||
|
|
|||
| require('./categories.spec'); | |||
There was a problem hiding this comment.
Unable to resolve path to module './categories.spec' import/no-unresolved
| import chai from 'chai'; | ||
| import chaiHttp from 'chai-http'; | ||
|
|
||
| import app from '../app'; |
There was a problem hiding this comment.
Unable to resolve path to module '../app' import/no-unresolved
Missing file extension for "../app" import/extensions
| @@ -0,0 +1,226 @@ | |||
| import chai from 'chai'; | |||
| import chaiHttp from 'chai-http'; | |||
There was a problem hiding this comment.
Unable to resolve path to module 'chai-http' import/no-unresolved
Missing file extension for "chai-http" import/extensions
| @@ -0,0 +1,226 @@ | |||
| import chai from 'chai'; | |||
There was a problem hiding this comment.
Unable to resolve path to module 'chai' import/no-unresolved
Missing file extension for "chai" import/extensions
benfluleck
deleted the
feature/153149477admin-should-be-able-to-upgrade-user-level
branch
PR implements the logic for the Administrator upgrading the user levels
Description
Description of Task to be completed?
Implemented admin functionality for the server side to manage the clients
Screenshots (if appropriate):
Any background context you want to provide?
Coverage on coveralls is at 93%
What are the relevant pivotal tracker stories?
Types of changes
Checklist: