Skip to content
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

Change where is_editor_of/has_editor associations are saved #521

Open
jcoyne opened this issue Apr 30, 2014 · 3 comments
Open

Change where is_editor_of/has_editor associations are saved #521

jcoyne opened this issue Apr 30, 2014 · 3 comments
Labels

Comments

@jcoyne
Copy link
Contributor

jcoyne commented Apr 30, 2014

The bidirectional has_and_belongs_to_many is_editor_of/has_editor requires twice as many writes to Fedora than are necessary. Instead we can have one side be has_and_belongs_to_many and the other can just be has_many. What say you curate devs?

@jeremyf
Copy link
Contributor

jeremyf commented May 1, 2014

@mstroming @rbalekai Can you weigh in on this? You did quite a bit of the work regarding this.

@mstroming
Copy link
Contributor

As long as the objects on both sides of the relationship know about the relationship (it's still bidirectional), I'm fine with whatever relationship type is used.

@rbalekai
Copy link
Contributor

rbalekai commented May 1, 2014

+1. we should be able to do the same for has_and_belongs_to_many is_editor_group_of/has_editor_group

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants