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

fix: Take into account parent's primaryColumn config when fetching fr… #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

danteBenitez
Copy link

…om polymorphic child

Before, on a child repository's options, we didn't handle the case in which there were multiple parents which had different primaryColumn names

Now we take such configurations into account. Specifically, we fetch the metadata associated with the parent we are visiting and read its primaryColumn. This allows for multiple parents to have different ID names when such situations arise.

…om polymorphic child

Before, on a child repository's options, we didn't handle the case in which there were multiple parents which had different `primaryColumn` names

Now we take such configurations into account. Specifically, we fetch the metadata associated with the parent we are visiting and read its `primaryColumn`. This allows for multiple parents to have different ID names when such situations arise.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant