Pnpjs: Implement usingCaching for search fluent API

Created on 27 Aug 2018  路  3Comments  路  Source: pnp/pnpjs

Category

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

Version

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

Expected / Desired Behavior / Question

Many of the PNP JS functions allow for the chained usingCaching. Although search has an internal caching layer, this is useful for reducing server roundtrips. Given the utility of search for producing rollups, usingCaching would be similarly beneficial for the fluent search API.

code complete enhancement

Most helpful comment

Yessir, works beautifully and now I've got a nice, snappy interface! Thanks for the fast response.

All 3 comments

The big issue here is we don't do caching currently for any non-get requests. This is because anything else is usually an action that changes something. For search this isn't true, BUT does mean we need a way to carve out a flag or something to allow caching non-get requests. Will see what this takes to implement, but can't promise we will add it. Def get what you mean though, thanks for the idea!

Had a look tonight, yep, we can do this. In fact, if you install beta v1.1.5-5 you can try it out now :) docs here. Let us know what you think!

Yessir, works beautifully and now I've got a nice, snappy interface! Thanks for the fast response.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

un-dres picture un-dres  路  3Comments

simkessy picture simkessy  路  3Comments

muraray picture muraray  路  3Comments

simonagren picture simonagren  路  3Comments

AJIXuMuK picture AJIXuMuK  路  3Comments