It is a little unexpected that you ``addFeature`` takes a success callback argument, but has a hard wired error function. It would be a bit more conventional to pass both callbacks as arguments. Alternatively, since ``addFeature`` is only called through one flow, hardcoding the success callback would be reasonable.