Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Natural key only find #8

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

Conversation

MGPalmer
Copy link
Contributor

Hoi, here's another feature I'm using.

FYI, when I use this in my project, I get warnings about missing key translations:

warn: ShippingCategory(655) not in keymap, referenced by Product(1069)#shipping_category_id
warn: Company(65) not in keymap, referenced by Product(1069)#company_id

But I can't reproduce this in the replicate tests. Since everything is still working, I'm still using this, but I don't know about you...

@rtomayko
Copy link
Owner

Thanks for this. I may change some of the option / methods names if that's cool.

@MGPalmer
Copy link
Contributor Author

Oh, absolutely.

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

Successfully merging this pull request may close these issues.

2 participants