Back in hello.rb, at the end of the route block, render the views/hello.erb template. What am I doing wrong?

Question

Please Help

hello.rb

require "sinatra"
set :bind, "0.0.0.0"

def page_content(title)
  File.read("pages/#{title}.txt")
rescue Errno::ENOENT
  return nil
end
get "/greet/:name" do
  @name = params[:name]
  @content = content_page(@name)
  erb :show
require 'views/hello.erb' 

get '/' do
  erb :views/hello
end

views/hello.erb

<p>Hello <%= @name %></p>

Answer 1 · 2017-05-25T05:00:07Z

May 25, 2017 5:00am

This confused me too, but I found that this completed the challenge:

erb :hello

Answer 2 · 2017-04-03T23:44:35Z

April 3, 2017 11:44pm

You may have misunderstood how the code is supposed to work, here. The goal is to let the user visit a path like /greet/Osvaldo or /greet/Jay, take the second portion of the path (Osvaldo or Jay), and display a greeting based only on that path (<p>Hello Osvaldo</p> or <p>Hello Jay</p>).

So this code you have is a good use of URL parameters. This is all you need to do to retrieve the name, you don't need to define or call a page_content method.

get "/greet/:name" do
  @name = params[:name]

But you need to add an end keyword to close the route's block. Also, get "/greet/:name" is the only route you need; you don't need a get '/' route.

I would recommend going back and reviewing the video on ERB templates. The info you need to render the template should be there.

Welcome to the Treehouse Community

Looking to learn something new?

Osvaldo Gonzalez

Osvaldo Gonzalez

Back in hello.rb, at the end of the route block, render the views/hello.erb template. What am I doing wrong?

2 Answers

Joan Estep

Joan Estep

Jay McGavren

Jay McGavren