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

tag identifyFeatures task output with parent layerId #462

Merged
merged 1 commit into from Feb 28, 2015

Conversation

@jgravois
Copy link
Contributor

@jgravois jgravois commented Feb 23, 2015

resolves #443
had to update existing tests. they're passing locally.

it seemed most sensible to me to just add this info whenever we're identifying features.

the benefit, (as we discussed on twitter), is that developers can now chain requests to do things like snagging attachments associated with a particular feature.

this is kind of a tangent, but it seems worth updating Util.responseToFeatureCollection() so that it returns an array of features whose order is the same as the input. right?

@jgravois jgravois force-pushed the jgravois:patch-443 branch from 9106440 to b0c709c Feb 23, 2015
updated tests
@jgravois jgravois force-pushed the jgravois:patch-443 branch from b0c709c to 5991a74 Feb 23, 2015
patrickarlt added a commit that referenced this pull request Feb 28, 2015
tag identifyFeatures task output with parent layerId
@patrickarlt patrickarlt merged commit bf17eb0 into Esri:master Feb 28, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jgravois jgravois deleted the jgravois:patch-443 branch Feb 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.