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

Let users influence the relationship mode in models #1001

Closed
wants to merge 2 commits into from
Closed

Let users influence the relationship mode in models #1001

wants to merge 2 commits into from

Conversation

bayleedev
Copy link
Member

Now the mode can be defined on the relationship or at runtime. Two tests.

@nateabele
Copy link
Member

Sorry, one other thing: maybe the default could get set in Database where the class is factoried?

@mariuswilms
Copy link
Member

Refs issue #453.

@nateabele
Copy link
Member

Anybody available to finalize this? It'd be great to get it in before the next release.

@mariuswilms mariuswilms added this to the 1.0 milestone Sep 29, 2014
@mariuswilms mariuswilms removed this from the 1.0 milestone Sep 29, 2014
@bayleedev bayleedev closed this Jun 14, 2015
@mariuswilms
Copy link
Member

@blainesch I guess you've been cleaning up branches. This PR still has some good code. We just couldn't get to it (I know it's about time...). Do you mind undoing the PR branch delete so this PR can be reopened?

@bayleedev
Copy link
Member Author

@DavidPersson I don't have a fork of lithium anymore, so it could get complicated. The "reopen" button is disabled. This sha1 is in the main repo now though due to the pr. Somebody could easily start from where I left off by checking out the sha1 or copy/pasting the patch file.

@mariuswilms
Copy link
Member

I'll handle this. There sure is a way.

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

Successfully merging this pull request may close these issues.

None yet

3 participants