Skip to content

Let users influence the relationship mode in models #1001

Closed
wants to merge 2 commits into from

3 participants

@blainesch
Union of RAD member

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

@nateabele
Union of RAD member

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

@davidpersson
Union of RAD member

Refs issue #453.

@nateabele
Union of RAD member

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

@davidpersson davidpersson added this to the 1.0 milestone Sep 29, 2014
@davidpersson davidpersson removed this from the 1.0 milestone Sep 29, 2014
@blainesch blainesch closed this Jun 14, 2015
@davidpersson
Union of RAD 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?

@blainesch
Union of RAD member

@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.

@davidpersson
Union of RAD 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
Something went wrong with that request. Please try again.