Skip to content

default count() param to true #241

@OlivierCuyp

Description

@OlivierCuyp

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:

this.urlComponents.oDataQueryParams.$count = isCount.toString();

That way you stay compatible with previous version and in the new version this param is optional.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions