Skip to content

MVP completed #128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 33 additions & 0 deletions api/actions/actions-middlware.js
Original file line number Diff line number Diff line change
@@ -1 +1,34 @@
// add middlewares here related to actions
const Action = require('./actions-model')

async function validateActionId(req, res, next){
try{
const action = await Action.get(req.params.id)
if(!action){
res.status(404).json({message: 'No actions were found with that ID'})
} else{
req.action = action
next()
}
}catch(err){
res.status(500).json({message: 'an error occured, action not found'})
}
}

async function validateAction (req, res, next){
const {project_id, description, notes, completed} = req.body
if(!project_id){
res.status(400).json({message: 'missing required project id'})
}
if(!notes || !notes.trim){
res.status(400).json({message: 'missing required project notes'})
} else{
req.project_id = project_id
req.description = description.trim()
req.notes = notes.trim()
req.completed = completed
next()
}
}

module.exports = {validateActionId, validateAction}
53 changes: 53 additions & 0 deletions api/actions/actions-router.js
Original file line number Diff line number Diff line change
@@ -1 +1,54 @@
// Write your "actions" router here!
const Actions = require("./actions-model")
const express = require("express")
const router = express.Router()
const {validateActionId, validateAction} = require("./actions-middlware")

router.get('/', async (req, res) => {
try{
const actions = await Actions.get()
res.status(200).json(actions)
} catch(err) {
next(err)
}
})

router.get("/:id", validateActionId, async (req, res, next) => {
try{
res.status(200).json(req.action)
} catch(err){
next(err)
}
})

router.post("/", async (req, res) => {
try{
const newAction = await Actions.insert(req.body)
console.log(newAction)
res.status(201).json(newAction)
} catch{
next(err)
}
})

router.put("/:id", validateActionId, validateAction, async (req, res,) => {
console.log(req.params.id)
const updatedAction = await Actions.update(req.params.id, {
project_id: req.project_id,
description: req.description,
notes: req.notes,
completed: req.completed
})
res.status(200).json(updatedAction)
})

router.delete("/:id", validateActionId, async (req, res, next) => {
try{
await Actions.remove(req.params.id)
res.json(res.Action)
} catch(err){
next(err)
}
})

module.exports = router
32 changes: 32 additions & 0 deletions api/projects/projects-middleware.js
Original file line number Diff line number Diff line change
@@ -1 +1,33 @@
// add middlewares here related to projects
const Projects = require("./projects-model")

async function validateProjectId(req, res, next){
try{
const {id} = req.params
const project = await Projects.get(id)
if(project){
req.params = project
next()
} else{
next({status: 404, message: 'project not found'})
}
}catch(err){
next(err)
}
}

async function validateProject(req, res,next){
const {name, description, completed} = req.body
if(!name || !name.trim()){
res.status(400).json({message: 'Name field required'})
} else if(!description || !description.trim()){
res.status(400).json({message: 'missing required description field'})
} else{
req.name = name.trim()
req.description = description.trim()
req.completed = completed
next()
}
}

module.exports = {validateProjectId, validateProject}
76 changes: 76 additions & 0 deletions api/projects/projects-router.js
Original file line number Diff line number Diff line change
@@ -1 +1,77 @@
// Write your "projects" router here!
const Projects = require("./projects-model")
const express = require("express")
const router = express.Router()
const {validateProjectId, validateProject} = require("./projects-middleware")


router.get("/", (req, res) => {
Projects.get()
.then(project => {
res.status(200).json(project)
})
.catch(err => {
res.status(500).json({message: 'The posts information could not be retrieved'})
})
})

router.get("/:id", validateProjectId, (req,res, next) => {
try{
res.status(200).json(req.params)
} catch(err){
next(err)
}
})

router.post('/', (req, res) => {
const newProject = req.body
Projects.insert(newProject)
.then(project => {
res.status(201).json(newProject)
})
.catch(error => {
console.log(error);
res.status(500).json({
message: 'Error adding the project',
})
})
})

router.put("/:id", validateProjectId, validateProject, (req,res, next) => {
const {name, description, completed} = req.body
if(!name || !description, !completed){
res.status(400).json({message: 'Project ID does not exist'})
} else{
Projects.update(req.params.id, req.body)
.then(() => {
return Projects.get(req.params.id)
})
.then(project => {
res.json(project)
})
.catch(next)
}
})

router.delete("/:id", validateProjectId, async (req, res, next) => {
try{
await Projects.remove(req.params.id)
res.json(res.Projects)
} catch(err){
next(err)
}
})

router.get("/:id/actions", validateProjectId, async (req, res, next) => {
Projects.getProjectActions(req.params.id)
.then(actions => {
if(actions.length > 0){
res.status(200).json(actions)
} else {
res.status(404).json((actions))
}
})
.catch(next)
})

module.exports = router
6 changes: 6 additions & 0 deletions api/server.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,15 @@
const express = require('express');
const server = express();
const projectsRouter = require("./projects/projects-router")
const actionsRouter = require("./actions/actions-router")

// Configure your server here
// Build your actions router in /api/actions/actions-router.js
// Build your projects router in /api/projects/projects-router.js
// Do NOT `server.listen()` inside this file!

server.use(express.json())
server.use("/api/projects", projectsRouter)
server.use("/api/actions", actionsRouter)

module.exports = server;
Binary file modified data/lambda.db3
Binary file not shown.
7 changes: 7 additions & 0 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,10 @@ I need this code, but don't know where, perhaps should make some middleware, don

Pull your server into this file and start it!
*/
const server = require('./api/server')

const PORT = process.env.PORT || 9000;

server.listen(PORT, () => {
console.log("server is running my guy")
})
Loading