This repository has been archived by the owner. It is now read-only.

Added SearchBuiltInSourceId class #421

Merged
merged 2 commits into from Apr 22, 2017

Conversation

Projects
None yet
4 participants
@phawrylak
Contributor

phawrylak commented Apr 19, 2017

Q A
Bug fix? [x]
New feature? [x]
New sample? [ ]

What's in this Pull Request?

I have added SearchBuiltInSourceId class for built in search result sources id constants. I also changed type of SearchResults.PrimarySearchResults from any to any[] - this allows to use array functions like map() without type casting.

@msftclas

This comment has been minimized.

Show comment
Hide comment
@msftclas

msftclas Apr 19, 2017

@phawrylak,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

msftclas commented Apr 19, 2017

@phawrylak,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@VesaJuvonen

This comment has been minimized.

Show comment
Hide comment
@VesaJuvonen

VesaJuvonen Apr 21, 2017

Contributor

thx Paweł, we'll process this during upcoming days.

Contributor

VesaJuvonen commented Apr 21, 2017

thx Paweł, we'll process this during upcoming days.

@patrick-rodgers patrick-rodgers merged commit 4b790e7 into SharePoint:dev Apr 22, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@patrick-rodgers

This comment has been minimized.

Show comment
Hide comment
@patrick-rodgers

patrick-rodgers Apr 22, 2017

Contributor

Looks great, thanks for the contribution!

Contributor

patrick-rodgers commented Apr 22, 2017

Looks great, thanks for the contribution!

@patrick-rodgers patrick-rodgers added this to the 2.0.5 milestone Apr 22, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.