Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Much more robust API (may help a lot of people) #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Commits on Jan 25, 2013

  1. Much more robust API

    0) Returns proper HTML response codes
    1) Much faster! (because it returns proper response codes browser
    doesn't wait!)
    2) Can't crash the app by sending bad data (checks what you send it)
    3) Makes sure the browser won't cache the data (problem solved)
    4) If it can't update or delete an item you now will be notified
    properly
    Dan Stroot committed Jan 25, 2013
    Configuration menu
    Copy the full SHA
    d9fd3f6 View commit details
    Browse the repository at this point in the history
  2. More Robust API

    Christophe: LOVE your tutorials! I have learned quite a bit! I wanted to
    give back by improving your wines api a bit -
    
    0) Returns proper HTML response codes
    1) Much faster! (because it returns proper response codes browser
    doesn't wait...)
    2) Can't crash the app by sending bad data (checks what you send it)
    3) Makes sure the browser won't cache the data (problem solved)
    4) If it can't update or delete an item you now will be notified
    properly
    5) res.send changed to res.json
    Dan Stroot committed Jan 25, 2013
    Configuration menu
    Copy the full SHA
    efffd87 View commit details
    Browse the repository at this point in the history
  3. Fixed dashboard so it doesn't count itself

    Dan Stroot committed Jan 25, 2013
    Configuration menu
    Copy the full SHA
    d1c8fb1 View commit details
    Browse the repository at this point in the history