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

Adding overloaded methods to handle Parent object reference using token and syncing the id on the child #9

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

Conversation

jefersonchaves
Copy link

Added possibility to set Parent using Field Token.
Added possibility to set Parent object along to Id on the child.
Added capability to retrieve field values.

Added possibility to set Parent object along to Id on the child.
Added capability to retrieve field values.
@jefersonchaves
Copy link
Author

Hi, @bobalicious : is this something worth adding? I can tidy it up if is something worth it.

@bobalicious
Copy link
Collaborator

It's along the lines of the kind of thing I was going to think about next, but it depends on how complete the solution should be - bear in mind the different ways that a parent relationship can now be set, using String names to automatically generate intermediate steps (e.g. Setting Account.Contact.Owner on an Opportunity). I haven't thought about it in-depth.

I guess a key question is - have you found it useful. If so, then it's probably worth exploring again. If not, then maybe not.

If the PR is re-raised, I'd welcome it, and I'll be happy to review and potentially merge - as long as it fits into the current API nicely.

@jefersonchaves
Copy link
Author

I actually have found this useful on one project, but to be honest it has been a long while.

I can take a look at updating this with the latest API, potentially submitting a new PR.

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.

2 participants