Skip to content

Latest commit

 

History

History
614 lines (513 loc) · 60.4 KB

File metadata and controls

614 lines (513 loc) · 60.4 KB

Code Review

npm-check

bootstrap                     NOTUSED?  Still using bootstrap?
bootstrap-star-rating         NOTUSED?  Still using bootstrap-star-rating?
eslint                        NOTUSED?  Still using eslint?
eslint-plugin-react           NOTUSED?  Still using eslint-plugin-react?
pg                            NOTUSED?  Still using pg?
pg-native                     NOTUSED?  Still using pg-native?
react-bootstrap-star-rating   NOTUSED?  Still using react-bootstrap-star-rating?

SQL

erd

Files

/db/models/category.js           ░
/db/models/index.js              ░░░░░
/db/models/product.js            ░░░
/db/models/reservation.js        ░░░░
/db/models/review.js             ░
/db/models/user.js               ░░░░░░░░░
/db/models/user.test.js          ░░░
/server/api.js                   ░░░
/server/auth.filters.js          ░░░
/server/auth.test.js             ░░░░░░░░░░░░
/server/categories.js            ░░
/server/products.js              ░░░░
/server/start.js                 ░░░░░░░░░
/server/userProfile.js           ░░░░░░░░░░░░░░
/server/userProfile.test.js      ░░░░
/server/users.js                 ░░░
/server/users.test.js            ░░░░░
/app/reducers/category.jsx       ░░░░░░░░
/app/reducers/index.jsx          ░░
/app/reducers/products.jsx       ░░░░░░░░░░░░░
/app/reducers/userProfile.js     ░░░░░░░░░░░░░░░
/app/components/Categories.jsx   ░░░░░░
/app/components/Login.jsx        ░░░░░░░░░░░░░░
/app/components/Login.test.jsx   ░░░░░░░░░
/app/components/Navbar.jsx       ░░░░
/app/components/PostProduct.jsx  ░░░░░░░░░░░░░░░░░░░░░░░░░░░░
/app/components/ProductView.jsx  ░░░░░░░░░░
/app/components/ProductsView.jsx ░░░░░░░░░
/app/components/Reservation.js   ░░░░░░░░░░
/app/components/Review.jsx       ░░░░░░
/app/components/ReviewsByUser.js ░░░░░░░░
/app/components/Sidebar.jsx      ░░░
/app/components/Signup.jsx       ░░░░░░░░░░░░░░░░░░
/app/components/UserProfile.jsx  ░░░░░░░░░░░░░
/app/components/WhoAmI.jsx       ░░░
/app/components/WhoAmI.test.jsx  ░░░░░░░

models

  • category.js

    • Linter

    • Human Review

  • index.js

    • Linter

    • Human Review

  • product.js

    • Linter

      • 5: 'Category' is assigned a value but never used.
    • Human Review

  • reservation.js

    • Linter

      • 9: A function with a name starting with an uppercase letter should only be used as a constructor.
      • 14: Missing space before =>.
      • 14: Missing space after =>.
      • 16: Missing space before value for key 'order'.
    • Human Review

  • review.js

    • Linter

    • Human Review

  • user.js

    • Linter

      • 36: Unexpected console statement.
    • Human Review

  • user.test.js

    • Linter

    • Human Review

routes

  • api.js

    • Linter

      • 3: 'db' is assigned a value but never used.
      • 4: A function with a name starting with an uppercase letter should only be used as a constructor.
      • 16: 'next' is defined but never used.
    • Human Review

  • auth.filters.js

    • Linter

      • 15: 'next' is defined but never used.
    • Human Review

  • auth.test.js

    • Linter

    • Human Review

  • categories.js

    • Linter

      • 5: A function with a name starting with an uppercase letter should only be used as a constructor.
      • 6: More than 2 blank lines not allowed.
    • Human Review

  • products.js

    • Linter

      • 2: A function with a name starting with an uppercase letter should only be used as a constructor.
    • Human Review

  • start.js

    • Linter

      • 24: Unexpected space between function name and paren.
      • 24: Unexpected space between function name and paren.
      • 54: Unexpected console statement.
      • 55: Unexpected console statement.
    • Human Review

  • userProfile.js

    • Linter

      • 3: 'db' is assigned a value but never used.
      • 4: 'Review' is assigned a value but never used.
      • 6: A function with a name starting with an uppercase letter should only be used as a constructor.
      • 10: 'next' is defined but never used.
      • 20: 'next' is defined but never used.
      • 34: 'next' is defined but never used.
      • 40: Missing space before =>.
      • 44: More than 2 blank lines not allowed.
      • 50: 'next' is defined but never used.
      • 52: 'SellerReview' is not defined.
      • 56: Missing space before =>.
      • 66: 'next' is defined but never used.
      • 69: 'RenterReview' is not defined.
      • 72: Missing space before =>.
      • 77: More than 2 blank lines not allowed.
    • Human Review

  • userProfile.test.js

    • Linter

      • 2: 'expect' is assigned a value but never used.
      • 3: 'db' is assigned a value but never used.
      • 4: 'User' is assigned a value but never used.
    • Human Review

  • users.js

    • Linter

      • 8: A function with a name starting with an uppercase letter should only be used as a constructor.
    • Human Review

  • users.test.js

    • Linter

      • 3: 'db' is assigned a value but never used.
      • 4: 'User' is assigned a value but never used.
    • Human Review

reducers

  • category.jsx

    • Linter

      • 7: More than 2 blank lines not allowed.
      • 21: Unexpected console statement.
      • 31: Identifier name 'a' is too short (< 2).
      • 31: Identifier name 'b' is too short (< 2).
    • Human Review

  • index.jsx

    • Linter

    • Human Review

  • products.jsx

    • Linter

      • 10: Missing space after =>.
      • 21: Missing space after =>.
      • 29: Missing space after =>.
      • 29: Missing space before =>.
      • 29: Missing space after =>.
      • 38: Unexpected console statement.
      • 44: Missing space before =>.
      • 44: Missing space after =>.
      • 44: Missing space after =>.
      • 53: Unexpected console statement.
      • 60: 'dispatch' is defined but never used.
      • 63: Unexpected console statement.
    • Human Review

  • userProfile.js

    • Linter

      • 31: Expected space(s) after "switch".
      • 50: Unnecessary semicolon.
      • 55: Missing space before =>.
      • 56: Unexpected console statement.
      • 59: Unexpected console statement.
      • 64: Missing space before =>.
      • 65: Unexpected console statement.
      • 68: Unexpected console statement.
      • 73: Missing space before =>.
      • 74: Unexpected console statement.
      • 77: Unexpected console statement.
      • 82: Missing space before =>.
      • 83: Unexpected console statement.
      • 86: Unexpected console statement.
      • 91: Missing space before =>.
      • 92: Unexpected console statement.
      • 95: Unexpected console statement.
    • Human Review

components

  • Categories.jsx

    • Linter

      • 13: Unexpected usage of singlequote.
      • 21: 'categories' is missing in props validation
      • 21: 'categories' is missing in props validation
      • 37: Unexpected space between function name and paren.
      • 37: Unexpected space between function name and paren.
      • 41: Unexpected space between function name and paren.
      • 41: Unexpected space between function name and paren.
    • Human Review

  • Login.jsx

    • Linter

      • 16: Identifier name 'e' is too short (< 2).
      • 17: Missing space before value for computed key 'e.target.name'.
      • 25: Property should be placed on a new line
      • 27: 'login' is missing in props validation
      • 36: Identifier name 'e' is too short (< 2).
      • 44: Identifier name 'e' is too short (< 2).
      • 48: Property should be placed on a new line
      • 50: Empty components are self-closing
      • 58: Identifier name 'e' is too short (< 2).
      • 60: 'user' is assigned a value but never used.
      • 75: 'ownProps' is defined but never used.
      • 78: More than 2 blank lines not allowed.
      • 81: 'ownProps' is defined but never used.
      • 84: Unexpected console statement.
      • 91: Too many blank lines at the end of file. Max of 1 allowed.
    • Human Review

  • Login.test.jsx

    • Linter

      • 53: Parsing error: Unexpected token )
    • Human Review

  • Navbar.jsx

    • Linter

      • 1: 'Component' is defined but never used.
      • 2: 'Login' is defined but never used.
      • 7: Component definition is missing display name
      • 12: Empty components are self-closing
      • 16: 'user' is missing in props validation
      • 16: A space is required before closing bracket
      • 20: Empty components are self-closing
    • Human Review

  • PostProduct.jsx

    • Linter

      • 6: More than 2 blank lines not allowed.
      • 25: Identifier name 'e' is too short (< 2).
      • 26: Missing space before value for computed key 'e.target.name'.
      • 28: More than 2 blank lines not allowed.
      • 32: Unexpected console statement.
      • 32: 'user' is missing in props validation
      • 36: Property should be placed on a new line
      • 38: 'post' is missing in props validation
      • 49: 'user' is missing in props validation
      • 49: 'user.id' is missing in props validation
      • 59: Identifier name 'e' is too short (< 2).
      • 67: Identifier name 'e' is too short (< 2).
      • 72: 'categories' is missing in props validation
      • 72: 'categories' is missing in props validation
      • 82: Identifier name 'e' is too short (< 2).
      • 90: Identifier name 'e' is too short (< 2).
      • 98: Identifier name 'e' is too short (< 2).
      • 106: Identifier name 'e' is too short (< 2).
      • 114: Identifier name 'e' is too short (< 2).
      • 123: Identifier name 'e' is too short (< 2).
      • 131: Identifier name 'e' is too short (< 2).
      • 134: Property should be placed on a new line
      • 136: Empty components are self-closing
      • 144: Identifier name 'e' is too short (< 2).
      • 146: 'user' is assigned a value but never used.
      • 161: 'ownProps' is defined but never used.
      • 167: More than 2 blank lines not allowed.
      • 170: 'ownProps' is defined but never used.
      • 173: Unexpected console statement.
      • 180: Too many blank lines at the end of file. Max of 1 allowed.
    • Human Review

  • ProductView.jsx

    • Linter

      • 1: 'Component' is defined but never used.
      • 2: 'store' is defined but never used.
      • 3: 'Link' is defined but never used.
      • 10: 'product' is missing in props validation
      • 11: Unexpected console statement.
      • 11: Strings must use singlequote.
      • 15: Unexpected usage of singlequote.
      • 18: A space is required before closing bracket
      • 19: Unexpected usage of singlequote.
      • 50: 'ownProps' is defined but never used.
      • 51: Unexpected console statement.
      • 51: Strings must use singlequote.
      • 57: 'ownProps' is defined but never used.
      • 57: Missing space after =>.
      • 58: Unexpected console statement.
      • 58: Strings must use singlequote.
      • 60: Missing space before =>.
      • 61: 'dispatch' is already declared in the upper scope.
      • 61: Missing space before =>.
      • 61: Missing space after =>.
    • Human Review

  • ProductsView.jsx

    • Linter

      • 1: 'Component' is defined but never used.
      • 2: 'store' is defined but never used.
      • 7: 'allProducts' is missing in props validation
      • 8: 'params' is missing in props validation
      • 8: 'params.categoryId' is missing in props validation
      • 13: Missing space before =>.
      • 13: Missing space after =>.
      • 16: Unexpected usage of singlequote.
      • 17: A space is required before closing bracket
      • 21: Unexpected usage of singlequote.
      • 35: Empty components are self-closing
      • 35: Unexpected usage of singlequote.
      • 50: 'ownProps' is defined but never used.
      • 56: 'ownProps' is defined but never used.
      • 56: Missing space after =>.
      • 58: 'dispatch' is already declared in the upper scope.
      • 58: Missing space before =>.
      • 58: Missing space after =>.
    • Human Review

  • Reservation.js

    • Linter

      • 12: 'product' is assigned a value but never used.
      • 12: 'selectedProduct' is missing in props validation
      • 13: 'user' is assigned a value but never used.
      • 14: Expected space(s) after "if".
      • 14: 'auth' is missing in props validation
      • 15: 'auth' is missing in props validation
      • 17: Expected space(s) after "else".
      • 17: Unnecessary semicolon.
      • 21: Expected space(s) after "return".
      • 24: Identifier name 'e' is too short (< 2).
      • 24: 'props' is not defined.
      • 31: Property should be placed on a new line
      • 33: Empty components are self-closing
      • 49: 'ownProps' is defined but never used.
      • 55: More than 2 blank lines not allowed.
      • 58: 'ownProps' is defined but never used.
    • Human Review

  • Review.jsx

    • Linter

      • 1: 'Component' is defined but never used.
      • 8: 'review' is missing in props validation
      • 9: More than 2 blank lines not allowed.
      • 12: Unexpected console statement.
      • 12: 'review' is missing in props validation
      • 14: Expected space(s) after "return".
      • 15: Unexpected usage of singlequote.
      • 26: 'color1' is missing in props validation
    • Human Review

  • ReviewsByUser.js

    • Linter

      • 3: 'axios' is defined but never used.
      • 12: Expected 'this' to be used by class method 'ratingChanged'.
      • 13: Unexpected console statement.
      • 19: Empty components are self-closing
      • 20: Empty components are self-closing
      • 21: Empty components are self-closing
      • 22: 'review' is missing in props validation
      • 22: Unexpected usage of singlequote.
      • 22: A space is required before closing bracket
      • 31: 'ownProps' is defined but never used.
      • 42: More than 2 blank lines not allowed.
      • 45: 'ownProps' is defined but never used.
    • Human Review

  • Sidebar.jsx

    • Linter

      • 2: 'Link' is defined but never used.
      • 6: Component definition is missing display name
      • 7: 'params' is missing in props validation
      • 12: 'location' is missing in props validation
      • 12: 'location.pathname' is missing in props validation
      • 12: 'location.pathname.startsWith' is missing in props validation
    • Human Review

  • Signup.jsx

    • Linter

      • 22: Identifier name 'e' is too short (< 2).
      • 23: Missing space before value for computed key 'e.target.name'.
      • 31: Property should be placed on a new line
      • 31: Identifier name 'e' is too short (< 2).
      • 41: Identifier name 'e' is too short (< 2).
      • 49: Identifier name 'e' is too short (< 2).
      • 57: Identifier name 'e' is too short (< 2).
      • 65: Identifier name 'e' is too short (< 2).
      • 69: Property should be placed on a new line
      • 71: Empty components are self-closing
      • 79: Identifier name 'e' is too short (< 2).
      • 101: 'ownProps' is defined but never used.
      • 104: More than 2 blank lines not allowed.
      • 107: 'ownProps' is defined but never used.
      • 114: Too many blank lines at the end of file. Max of 1 allowed.
    • Human Review

  • UserProfile.jsx

    • Linter

      • 5: 'Link' is defined but never used.
      • 6: 'AccountInfoFormContainer' is defined but never used.
      • 21: 'routeParams.id' is missing in props validation
      • 21: 'routeParams' is missing in props validation
      • 23: 'getUserInfo' is missing in props validation
      • 24: 'getRentedTransactions' is missing in props validation
      • 25: 'getSoldTransactions' is missing in props validation
      • 26: 'getAsRenterRatings' is missing in props validation
      • 27: 'getAsSellerRatings' is missing in props validation
      • 30: Expected 'this' to be used by class method 'aggregateRatings'.
      • 31: Expected space(s) after "if".
      • 32: Missing space before =>.
      • 40: 'aggregateRatingAsRenter' is assigned a value but never used.
      • 40: 'rentingRatings' is missing in props validation
      • 41: 'aggregateRatingAsSeller' is assigned a value but never used.
      • 41: 'sellingRatings' is missing in props validation
      • 42: 'id' is assigned a value but never used.
      • 42: 'routeParams.id' is missing in props validation
      • 42: 'routeParams' is missing in props validation
      • 48: Empty components are self-closing
      • 56: 'children' is missing in props validation
      • 56: 'children' is missing in props validation
      • 64: 'ownProps' is defined but never used.
      • 75: 'ownProps' is defined but never used.
    • Human Review

  • WhoAmI.jsx

    • Linter

      • 4: 'user' is missing in props validation
      • 4: 'logout' is missing in props validation
      • 12: Empty components are self-closing
      • 21: Unexpected space between function name and paren.
      • 21: Unexpected space between function name and paren.
      • 24: Unexpected space between function name and paren.
      • 24: Unexpected space between function name and paren.
    • Human Review

  • WhoAmI.test.jsx

    • Linter

      • 17: Arrow function should not return assignment.
      • 18: A space is required before closing bracket
      • 31: Expected an assignment or function call and instead saw an expression.
      • 40: 'dispatch' is assigned a value but never used.
      • 42: 'state' is already declared in the upper scope.
      • 44: A space is required before closing bracket
    • Human Review

Route Tree

router                                                                                                         
 ├── query                                         *                                                           
 ├── expressInit                                   *                                                           
 ├── volleyball                                    *                                                           
 ├── _cookieSession                                *                                                           
 ├── urlencodedParser                              *                                                           
 ├── jsonParser                                    *                                                           
 ├── initialize                                    *                                                           
 ├── authenticate                                  *                                                           
 ├── serveStatic                                   *                                                           
 ├── serveStatic                                   *                                                           
 ├── router                                        /^\/api\/?(?=\/|$)/                                         
 │   router                                                                                                    
 │    ├── bound dispatch                           /heartbeat                                                  GET
 │    │    └── <anonymous>                         /                                                           GET
 │    │   
 │    ├── router                                   /^\/auth\/?(?=\/|$)/                                        
 │    │   router                                                                                               
 │    │    ├── bound dispatch                      /whoami                                                     GET
 │    │    │    └── <anonymous>                    /                                                           GET
 │    │    │   
 │    │    ├── bound dispatch                      /:strategy/login                                            POST
 │    │    │    └── <anonymous>                    /                                                           POST
 │    │    │   
 │    │    └── bound dispatch                      /logout                                                     POST
 │    │         └── <anonymous>                    /                                                           POST
 │    │        
 │    │   
 │    ├── router                                   /^\/users\/?(?=\/|$)/                                       
 │    │   router                                                                                               
 │    │    ├── bound dispatch                      /                                                           GET
 │    │    │    ├── <anonymous>                    /                                                           GET
 │    │    │    └── <anonymous>                    /                                                           GET
 │    │    │   
 │    │    ├── bound dispatch                      /                                                           POST
 │    │    │    └── <anonymous>                    /                                                           POST
 │    │    │   
 │    │    └── bound dispatch                      /:id                                                        GET
 │    │         ├── mustBeLoggedIn                 /                                                           GET
 │    │         └── <anonymous>                    /                                                           GET
 │    │        
 │    │   
 │    ├── router                                   /^\/categories\/?(?=\/|$)/                                  
 │    │   router                                                                                               
 │    │    └── bound dispatch                      /                                                           GET
 │    │         └── <anonymous>                    /                                                           GET
 │    │        
 │    │   
 │    ├── router                                   /^\/products\/?(?=\/|$)/                                    
 │    │   router                                                                                               
 │    │    ├── bound dispatch                      /                                                           GET
 │    │    │    └── <anonymous>                    /                                                           GET
 │    │    │   
 │    │    ├── bound dispatch                      /                                                           POST
 │    │    │    └── <anonymous>                    /                                                           POST
 │    │    │   
 │    │    └── bound dispatch                      /:productId                                                 GET
 │    │         └── <anonymous>                    /                                                           GET
 │    │        
 │    │   
 │    ├── router                                   /^\/userProfile\/?(?=\/|$)/                                 
 │    │   router                                                                                               
 │    │    ├── bound dispatch                      /id/:userId                                                 GET
 │    │    │    └── <anonymous>                    /                                                           GET
 │    │    │   
 │    │    ├── bound dispatch                      /reservations/asRenter/:userId                              GET
 │    │    │    └── <anonymous>                    /                                                           GET
 │    │    │   
 │    │    ├── bound dispatch                      /reservations/asSeller/:userId                              GET
 │    │    │    └── <anonymous>                    /                                                           GET
 │    │    │   
 │    │    ├── bound dispatch                      /ratings/asRenter/:userId                                   GET
 │    │    │    └── <anonymous>                    /                                                           GET
 │    │    │   
 │    │    └── bound dispatch                      /ratings/asSeller/:userId                                   GET
 │    │         └── <anonymous>                    /                                                           GET
 │    │        
 │    │   
 │    ├── <anonymous>                              *                                                           
 │    └── <anonymous>                              *                                                           
 │   
 └── bound dispatch                                /*                                                          GET
      └── <anonymous>                              /                                                           GET