brehtyn / business-routing-saas Goto Github PK

Still am unsure as to where to put the pending and holding count. In the header.ejs file at line 182 I see a post_count but am uncertain if that is total posts or only pending or only holding posts. It'd be cool if Brentyn saw this and could comment in which file and what line he wants the pending and holding count

Get all the forms in the header to work. The only one in there right now is the machines form so must include buildings. The form is not being sent anywhere so no http request is ever being sent so it will literally never do anything at all. It's just a button that you can click without being attached to anything. Fix this and do the same for buildings

the location object cannot handle more than one weekday in the drop_days attribute & still display on dashboard

It looks like machines associated with this specific location are supposed to be populated at the bottom

Express connect.session() won't scale past a single process because MemoryStore is not setup. Should be able to fix this using Redis on Heroku

@Dvent1123

clicking "delete user" on the delete user index page doesnt redirect to the delete user page, I suspect because of an permissions error

The /buildings/show route should get 3 arrays. One for each type of tech service (full, contract, or drop) and separate them based on this. Easiest way would be to make partials for each type of contract and include it in the buildings/show file. Need to ensure that all the buildings in the db are changed to the same syntax (FULL, CONTRACT, DROP)

Editing redirects to the main machines page with no changes in mongodb nor the application

No issues with creating a new machines though

@Dvent1123

complete call should move post into a history folder (or delete for now

@Dvent1123

it appears that machines will not update their location if their location's name changes, perphaps mongodb?

Couldn't connect to any server and this is an unhandled exception. Should be an easy fix with async function or .catch()

comment edit needs to be structured and styled so that authorization level of tech and higher can change title

A little confused on what the difference is between posting a comment which is an "updated issue" and editing the post itself which is the "main issue"? Wouldn't the updated issue now be the main issue? I feel like the flow of these components could be updated

Include the forms for the user and the buildings in the header. The machines form is already there so just need to include ones for user and building

Same thing as buildings and machines in header. Need the form in this one though to be able to create a new post. Input functionality here at least

should include basic validation incase input area = null and other usecases

Need to style the page for the individual user. users/show specifically

an incorrect username or password caues an error screen

@Dvent1123

trying to change the shortname & type of a location doesnt appear to be working for me

@Dvent1123
error occurs when clicking "delete machine" on the machines/index.ejs , not sure when introduced

Error: Failed to lookup view "machines/deleteMachine" in views directory "C:\Users\admin\Desktop\Business-Service-Program/views" at Function.render (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\application.js:580:17) at ServerResponse.render (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\response.js:1012:7) at ServerResponse.res.render (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express-ejs-layouts\lib\express-layouts.js:77:18) at C:\Users\admin\Desktop\Business-Service-Program\routes\machines.js:44:13 at Layer.handle [as handle_request] (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\layer.js:95:5) at next (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\route.js:137:13) at authDeleteProject (C:\Users\admin\Desktop\Business-Service-Program\routes\machines.js:139:5) at Layer.handle [as handle_request] (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\layer.js:95:5) at next (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\route.js:137:13) at checkAuthenticated (C:\Users\admin\Desktop\Business-Service-Program\permissions\basicAuth.js:3:16) at Layer.handle [as handle_request] (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\layer.js:95:5) at next (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\route.js:137:13) at Route.dispatch (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\route.js:112:3) at Layer.handle [as handle_request] (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\layer.js:95:5) at C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\index.js:281:22 at Function.process_params (C:\Users\admin\Desktop\Business-Service-Program\node_modules\express\lib\router\index.js:335:12)

In index.ejs on line 53 there is a link that is href'd back to the index. Figure out a way to attach each machine with the post so that the link will take you to that machine based on if it matches the location. One line below on line 54 do the same for that line so that it can find the machine based on the asset number.

Fix user index route. When the user index route is clicked on the dashboard the user page cannot get the route. Figure out why the page is not rendering. I have no idea. Maybe check user permissions possibly. Put in try catch and console log the error to see why it's not rendering correctly.