-
-
Notifications
You must be signed in to change notification settings - Fork 11
Add support for scopes #17
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
base: main
Are you sure you want to change the base?
Conversation
I like it. Thanks. I'm curious whether we can chain scopes off of where.not? For example, does where.not.deleted work here? I can check this myself later but if you're at a terminal and willing to try it out that may speed things up! |
- where.not("#{field}": nil)
+ where.not.send(field) It works. Do you prefer this? EDIT: or maybe you meant remove the |
Yeah my thought is to not include the not_ scope, since it seems to be pretty easy to do in app code. |
Fine. Let's rename the option |
Hey @calebhearth let me know if there's anything else I can do to get this merged :) |
Close #11