Timeline for AngularJS: Http API Service - any improvements / remarks?

7 events

when toggle format	what		by	license	comment
May 23, 2017 at 12:40	history	edited	CommunityBot		replaced http://stackoverflow.com/ with https://stackoverflow.com/
Nov 30, 2015 at 12:51	comment	added	kbd		Minor side note regarding 6: I'm not a fan of lowerCamelCase for functions/methods, because that makes callbacks confusing. Hard to see if you're passing a variable or function. I prefer the .net naming rules.
Nov 30, 2015 at 12:26	comment	added	kbd		I'll look into it when able, thanks for the information!
Nov 30, 2015 at 12:14	history	edited	Roamer-1888	CC BY-SA 3.0	deleted 6 characters in body
Nov 30, 2015 at 12:07	comment	added	Roamer-1888		Bind statements often look wrong. I've used them quite a bit and still have to concentrate. The thing to remember is that `.bind()` is not a call but something that returns a function. Here, a `null` context is provided (doRequest() and doViewRequest() don't use `this` therefore don't need a context), and "get" etc are "bound-into" the returned function as its first param. Further param(s) are passed when the returned function is eventually called for real, and don't need to appear in these `.bind()` statements.
Nov 30, 2015 at 7:08	comment	added	kbd		All very good points, thanks! A question with point 7, shouldn't there be a data parameter in the `bind()` for post/put?
Nov 29, 2015 at 18:31	history	answered	Roamer-1888	CC BY-SA 3.0