-
Notifications
You must be signed in to change notification settings - Fork 232
Labels
Description
Feature Request
Is your feature request related to a problem? Please describe
I don't understand why we should specify true in the .count() method.
If you call, it means you want it.
If you don't specify it, you get this error:
Cannot read property 'toString' of undefined at GraphRequest.count
PS: I'm not using typescript and I'm not the only one ;)
Describe the solution you'd like
Default the param to true here:
msgraph-sdk-javascript/src/GraphRequest.ts
Line 493 in 7baaf98
| this.urlComponents.oDataQueryParams.$count = isCount.toString(); |
That way you stay compatible with previous version and in the new version this param is optional.