Create Pizza App

[ASPhillips8]

Display list of pizzerias with link to form to create a new pizzeria. New Pizzeria creation has validations for name uniqueness and address. Pizzerias show page has link to pizza create form. Pizza has validation for name and description. Creation of successful pizza associates with pizzeria and redirects to pizza show page displaying name, description and link to the pizzeria it belongs to. Created list of pizza with link to pizza show page on each of the pizzeria show page.

[stephenmckeon]

Nice work, just a few small things, but lookin good otherwise!

[stephenmckeon]

Try to avoid pushing up changes like this for the real deal. Only commit changes that you actually care about. This means actually taking the time to check what you're staging and committing.

[stephenmckeon]

If you're not going to handle the notice, then don't include it. Right now, it doesn't look like that message is ever being displayed.

[stephenmckeon]

Suggested change

	params.require(:pizza).permit(:name,:description, :pizzeria_id)
	params.require(:pizza).permit(:name, :description, :pizzeria_id)

[stephenmckeon]

WATCH YOUR SPACING!!!!!!

[stephenmckeon]

Suggested change

	<h2>Tell us about this pizza you had at <%= @pizzeria.name %></h2>
	<% if @pizza.errors.any? %>
	<div id="error_explanation">
	<h2><%= pluralize(@pizza.errors.count, "error") %> prohibited this pizza from being saved:</h2>
	<ul>
	<% @pizza.errors.full_messages.each do |message| %>
	<li><%= message %></li>
	<% end %>
	</ul>
	</div>
	<% end %>
	<%= form_with model:[@pizzeria, @pizza], local: true do |form| %>
	<%= form.hidden_field :pizzeria_id, value: @pizzeria.id %>
	<%= form.label :name %>
	<%= form.text_field :name %>
	<%= form.label :description %>
	<%= form.text_area :description %>
	<%= form.submit "Create Pizza" %>
	<% end %>
	<h2>Tell us about this pizza you had at <%= @pizzeria.name %></h2>
	<% if @pizza.errors.any? %>
	<div id="error_explanation">
	<h2><%= pluralize(@pizza.errors.count, "error") %> prohibited this pizza from being saved:</h2>
	<ul>
	<% @pizza.errors.full_messages.each do |message| %>
	<li><%= message %></li>
	<% end %>
	</ul>
	</div>
	<% end %>
	<%= form_with model:[@pizzeria, @pizza], local: true do |form| %>
	<%= form.hidden_field :pizzeria_id, value: @pizzeria.id %>
	<%= form.label :name %>
	<%= form.text_field :name %>
	<%= form.label :description %>
	<%= form.text_area :description %>
	<%= form.submit "Create Pizza" %>
	<% end %>

[stephenmckeon]

Suggested change

	<p>
	<%= link_to "Tell us about a great Pizza you had at #{@pizzeria.name}", new_pizzeria_pizza_path(@pizzeria) %>
	</p>
	<h3>Great eats from <%= @pizzeria.name %>:</h3>
	<% @pizzeria.pizzas.each do |pizza| %>
	<p> <%= link_to pizza.name, pizzeria_pizza_path(pizza)%></p>
	<% end %>
	<p>
	<%= link_to "Tell us about a great Pizza you had at #{@pizzeria.name}", new_pizzeria_pizza_path(@pizzeria) %>
	</p>
	<h3>Great eats from <%= @pizzeria.name %>:</h3>
	<% @pizzeria.pizzas.each do |pizza| %>
	<p> <%= link_to pizza.name, pizzeria_pizza_path(pizza)%></p>
	<% end %>

[stephenmckeon]

Suggested change

	create_table :pizzas do |t|
	t.string :name
	t.text :description
	t.references :pizzeria
	t.timestamps
	end
	create_table :pizzas do |t|
	t.string :name
	t.text :description
	t.references :pizzeria
	t.timestamps
	end

[stephenmckeon]

Nice change, but watch the SPACING!

Suggested change

	resources :pizzerias, only: [:new, :create, :show, :index] do
	resources :pizzas, only: [:new, :create, :show]
	end
	resources :pizzerias, only: [:new, :create, :show, :index] do
	resources :pizzas, only: [:new, :create, :show]
	end