I reviewed your app and everything looks good. The only thing that I would mention is the creation and use of all these instance variables that need to get passed to the views. You could just pass the one @omdb_data object to the show.erb view page and dereference the specific items there. In show.erb you could write <h4><strong> Plot: </strong> <%=@omdb_data["Plot"] %> </h4>. As you have seen in Rails, we pass the "@todos" collection of todo items or "@todo" the one item to the view and then pluck out the details of the object in the views.