Code review

app/assets/javascripts/gifts.js.coffee

@@ -3,6 +3,8 @@
 # You can use CoffeeScript in this file: http://coffeescript.org/
 
 
+# Should be consitat with the project and use either all JS
+# or all Coffeescript
 $ ->
   $('.gift').hover (event) ->
     console.log("hover triggered")

app/assets/javascripts/jazController.js

@@ -18,7 +18,9 @@ Controller.prototype = {
         url: "/gifts.json",
         type: "GET",
         //'for' loop extracts gift data from the JSON & creates gift objects
+        // consider moving this anonymous function into a named function
         success: function(data) {
+          // don't commit debugging statements
           console.log(data);
           self.gifts = [];
           for (var i=0; i<data.length; i++) {
@@ -61,6 +63,8 @@ Controller.prototype = {
     // send the html from the template to the view
     this.view.showGiftDetails(html);
 
+    // don't commit commented out code
+    // use git and Github to refer to old code instead.
     // bind events for the save and delete buttons
     // $(this.view.saveButtonSelector).on("click", this.updateGiftDetails.bind(this));
     // $(this.view.deleteButtonSelector).on("click", this.deleteGift.bind(this));

app/controllers/dashboard_controller.rb

@@ -1,4 +1,5 @@
 class DashboardController < ApplicationController
+  # Minor niggle but try to remove trailing whitespace
 
   def index 
   end 

app/controllers/profiles_controller.rb

@@ -1,8 +1,11 @@
 class ProfilesController < ApplicationController
   def show
+    # Don't commit commented out code
     # @user = User.find_by_profile_name(params[:id])
     @user = User.find(params[:id])
     if @user 
+      # By default the association will give you all gifts for this user so you
+      # don't need to call all after.
       @gifts = @user.gifts.all 
       render action: :show 
     else 

app/helpers/dashboard_helper.rb

@@ -1,2 +1,3 @@
+# Don't commit empty files. Any empty helpers should just be deleted.
 module DashboardHelper
 end

app/models/gift.rb

@@ -1,4 +1,8 @@
 class Gift < ActiveRecord::Base
+  # Similar to whitespace get the whole team on one type of tab
+  # I would suggest 2 spaces for a tab. Seems like a small thing
+  # but different editors show tabs differently and it messes
+  # with indentation making the code harder to read.
 	validates :title, presence: true
   validates :description, presence: true
 

app/models/request.rb

@@ -1,4 +1,6 @@
 class Request < ActiveRecord::Base
   belongs_to :user
   belongs_to :gift
+  # Should add validations to make sure the
+  # foreign keys have been added
 end

app/views/dashboard/index.html.erb

@@ -16,6 +16,7 @@
       <% current_user.gifts.each do | gift| %>
         <div class="row">
           <div class="col-xs-2 ">
+            <!-- should be using link_to helper here -->
             <a href="/gifts/<%= gift.id %>">
               <%= gift.title %>
             </a>
@@ -41,4 +42,4 @@
             <% end %>      
 
     </div>
-</div>
+</div>

app/views/dashboard/requested.html.erb

@@ -14,6 +14,7 @@
   <ul class="nav nav-tabs" role="tablist">
     <li class="active"><a href="#my-gifts" role="tab" data-toggle="tab">My Gifts</a></li>
     <li><a href="#my-requests" role="tab" data-toggle="tab">My Requests</a></li>
+    <!-- broken link tag, should use link_to. Also could use helper method here to create whole li tags -->
     <li>a href="#profile" role="tab" data-toggle="tab">Profile</a></li>
   </ul>
 
@@ -43,4 +44,4 @@
             <% end %>      
 
     </div>
-</div>
+</div>

config/routes.rb

@@ -18,6 +18,9 @@
   # Jaz JS learning objectives:
   get 'gifts/:id/jaz', to: 'gifts#jaz'
 
+  # Remove commented out code. I know this is generated by Rails but good to remove
+  # commented out code ASAP as it just adds noise and hangs around long after it
+  # is intended to.
   # The priority is based upon order of creation: first created -> highest priority.
   # See how all your routes lay out with "rake routes".