• 
      

    Centralize logic and data into an Application/ApplicationView.

    Review Request #7527 — Created July 16, 2015 and submitted

    Information

    student-sonar
    master
    20a90dc...

    Reviewers

    This moves most of the code out of the main.js file into a model/view
    responsible for running the application. This also moves the header into a
    view, rather than rendering it from the hbs templates.

    Flipped through the different views. Saw that everything still worked.

    reviewbot
    1. Tool: PEP8 Style Checker
      Ignored Files:
          lib/frontend/application.js
          lib/frontend/header-view.js
          views/student-list.handlebars
          lib/frontend/main.js
          lib/frontend/application-view.js
          lib/frontend/index-view.js
          lib/frontend/admin-view.js
          views/layouts/main.handlebars
      
      
      
      Tool: Pyflakes
      Ignored Files:
          lib/frontend/application.js
          lib/frontend/header-view.js
          views/student-list.handlebars
          lib/frontend/main.js
          lib/frontend/application-view.js
          lib/frontend/index-view.js
          lib/frontend/admin-view.js
          views/layouts/main.handlebars
      
      
    2. 
        
    brennie
    1. Ship It!
    2. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to master (1ec9192)