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

Add support for target keys #25

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ricardogama
Copy link

This PR adds support for target keys that was introduced on bookshelf/bookshelf#1397.

@ricardogama ricardogama force-pushed the feature/target-keys branch 6 times, most recently from 47a2a56 to df80ca2 Compare October 12, 2016 07:35
@ricardogama ricardogama force-pushed the feature/target-keys branch 3 times, most recently from 1429800 to 9aea9d8 Compare November 10, 2016 23:31
@pepjo
Copy link

pepjo commented Mar 30, 2017

I've read here #9 (comment) that this cannot be merged until the next version of bookshelf has been released. To which version where you refering to? It seems that there have been some new versions between this dates.

@ricardogama
Copy link
Author

I guess that I was assuming bookshelf/bookshelf#1419 was going to be merged, without that fix this feature is incomplete.

Although it might be outdated you can use that Bookshelf's PR branch and this one and try the feature out.

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

Successfully merging this pull request may close these issues.

2 participants