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

change fetchRecords to be a getFoos with default accountId and ids #121

Closed
brong opened this issue Jul 19, 2017 · 1 comment
Closed

change fetchRecords to be a getFoos with default accountId and ids #121

brong opened this issue Jul 19, 2017 · 1 comment

Comments

@brong
Copy link
Member

brong commented Jul 19, 2017

3.10.2 getFooUpdates

fetchRecords & fetchRecordProperties

Instead of two separate parallel properties, how about defining fetchRecords as having the same structure as a getFoos object, but with appropriate defaults for accountId and ids. This way, extensions to the getFoos object naturally apply to the fetchRecords value object.

@neilj
Copy link
Member

neilj commented Oct 23, 2017

This was resolved by ac46f53 as there are no longer fetchRecords arguments to make implicit calls, but instead you can make explicit calls using back references for arguments to use the result of a previous one.

@neilj neilj closed this as completed Oct 23, 2017
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

No branches or pull requests

2 participants