Pnpjs: Graph skip property isn't working on a GraphQueryableCollection

[pkmelee337]

Category

• [ ] Enhancement
• [ x] Bug
• [ ] Question
• [ ] Documentation gap/issue

Version

Please specify what version of the library you are using: [ 1.2.3 ]

Please specify what version(s) of SharePoint you are targeting: [ SPO ]

Expected / Desired Behavior / Question

When specified as followed, I expect the "skip" parameter to get added to the query.

let view = new GraphQueryableCollection(graph.me, "people");
view = view.skip(12)

Observed Behavior

The skip parameter was not added. The following code works perfectly though:

let view = new GraphQueryableCollection(graph.me, "people");
view.query.set("skip", "12")

Steps to Reproduce

Add the code above 👍

[patrick-rodgers]

Amazed this wasn't found earlier. Copy/paste bug as we didn't update skip method to add skip rather it adds "top". Fixing.

[patrick-rodgers]

Fixed in #333.