-
Notifications
You must be signed in to change notification settings - Fork 84
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
Added pluralization support #20
base: master
Are you sure you want to change the base?
Conversation
Changes Unknown when pulling 677a871 on donmasakayan:pluralization_support into * on waterlock:master*. |
1 similar comment
Changes Unknown when pulling 677a871 on donmasakayan:pluralization_support into * on waterlock:master*. |
Wouldn't you be be able to access the |
Yes, that would be much more ideal. I was just implementing the same thing in waterlock-facebook-auth, line 27 for consistency. As such, both would need to be changed. |
Did I not do this? Maybe I didn't create a PR pretty sure I have. If you can wait until tomorrow morning I can create a On Sunday, 22 February 2015, donmasakayan notifications@github.com wrote:
-------------------- w:// t: 07508 215 459 -------------------- |
Thanks a bunch @donmasakayan but @Globegitter has a good point, this seems a little too "insider baseball" can we not solve this with blueprints? |
ok, so turns out I am just going up to waterlock.js: https://github.com/waterlock/waterlock-facebook-auth/blob/master/lib/waterlock-facebook-auth.js#L9 How would I go about doing this using blueprints? If someone can give me some pointers I'll create a PR for fb and local auths |
@wayne-o I have accessed the sails config before, but can't remember anymore how I did it. There is a global Hope this helps a bit. |
Allows for
auths/
instead ofauth/
when pluralization is enabled.