Skip to content

finished grading #3

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.

Sign up for GitHub

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
rvanderheyde wants to merge 1 commit into
base: master
Choose a base branch
from
Open

finished grading #3

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
7 changes: 5 additions & 2 deletions lessons/05-client-jquery-ajax/ZG_burgerApp/public/javascripts/ingredients.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ $ingredientForm.submit(function(event) {

var name = $ingredientForm.find("[name='name']").val();
var price = $ingredientForm.find("[name='price']").val();
//might want to check that name and price are not empty.

$.post("postIngredient", {
name: name,
Expand All @@ -33,7 +34,9 @@ $ingredientForm.submit(function(event) {
});

//runs when the user clicks the out of stock button for an ingredient
//You can also use .click it is the same thing.
$("body").on("click", ".outOfStock", function(){ //had to use .on() method for dynamically added ingredients
//you take advantage of html classes here also, but this works.
var isDisabled = !($(this).html() === 'Out of Stock'); //I really want to know a better way of doing this than looking at
//the raw html but I looked into adding my own attributes by modifying the doctype html and I found this one easier
if (isDisabled){ //if the ingredient was already disabled and they click the button, enable the ingredient
Expand All @@ -47,7 +50,7 @@ $("body").on("click", ".outOfStock", function(){ //had to use .on() method for
$.post("postStock", { //sends a post request to the server with the isDisabled boolean to determine what to set, and the id
isDisabled: isDisabled,
ingredientId: $(this).parent().attr("id")
})
}) //generally good to do things when the request returns, so that the user doesn't think something happened when it actually errored.
.done() //do nothing on client side when done, could have also had the css changes happen in on success. don't know which is best practice
.error(onError);
});
Expand All @@ -66,7 +69,7 @@ $("body").on("click", ".Edit", function(){
var newName = window.prompt("Please enter the new ingredient name: ", $(this).siblings("#name").html().slice(12));
if (newName !== ""){
var newPrice = window.prompt("Please enter the new price for " + newName, $(this).siblings("#price").html().slice(7));
if (newPrice !== ""){}
if (newPrice !== ""){
var ingredientId = $(this).parent().attr("id");
$.post("postEdit", {
newName: newName,
Expand Down
4 changes: 3 additions & 1 deletion lessons/05-client-jquery-ajax/ZG_burgerApp/routes/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
//you probably want to separate this into multiple files.

var Ingredient = require('../models/ingredientModel.js');
var Order = require('../models/orderModel.js');
var mongoose = require('mongoose');
Expand All @@ -9,7 +11,7 @@ var ingredients = function(req, res){
//with all of the ingredients
Ingredient.find({}, function(err, ingredients){
if (err){
res.sendStatus(500); //sends status to browser if error
res.status(500).send(err); //sends status to browser if error
console.log("Error: ", err);
} else {
var data = {ingredients: ingredients};
Expand Down