-
Notifications
You must be signed in to change notification settings - Fork 95
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
has_and_belongs_to_many_with_tenant does not include 3 arguments #206
Comments
Laykou
referenced
this issue
Sep 5, 2023
* Adds has_and_belongs_to_many featurewith tenant * Fixes static code analysis * Parameterizes field name * Parameterizes field name
Laykou
added a commit
to Laykou/activerecord-multi-tenant
that referenced
this issue
Sep 5, 2023
Was this parameter added to ActiveRecord in the last few versions? |
I see this has been already there even 7 years ago |
serprex
pushed a commit
that referenced
this issue
Sep 7, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Original implementation of
has_and_belongs_to_many
is like:However
has_and_belongs_to_many_with_tenant
only includes these arguments:scope
argument is missing here. Therefore when used like this on a model:then it fails with:
The text was updated successfully, but these errors were encountered: