-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
$resource #86
Comments
$resource is great, but this is a style guide. What type of style would you like to see in regards to it? |
I will say somewhere in Data Services, but you are right, I forgot for On Fri, Oct 10, 2014 at 9:47 PM, John Papa notifications@github.com wrote:
|
No worries. There are a lot of tips like this I think are useful. If anything, when you add them to issues like this other people will discover them. But I am compiling a list of these and finding a good place to host them too. Stay tuned :) |
Opening an old post, I hope you would respond anyway. The easy way (convention ?) to use $resource is to define one factory by resource URI, for instance :
creating the following files : In your styleguide, you recommend to put data services in a single service "data.service.js". Does it still apply if I use the "$resource as data service" style ? |
@sylvain-brouillat in my case I've decided to have all my data services on a 'data' module and for each context I have .dataservice.js. The reason I do this is because each context contains a lot of operations and feels more cleaner to separate them. Does this naming sounds good to you? Also this makes all my jasmine tests a lot more organized and clear. Basically I fallow the same logical tree to organize them. |
No word on using $resource! They made it easy to access repositories, but we heard nothing about that!
The text was updated successfully, but these errors were encountered: