search

Esri / arcgis-rest-js

compact, modular JavaScript wrappers for the ArcGIS REST API
https://developers.arcgis.com/arcgis-rest-js/
Apache License 2.0
347 stars 120 forks source link

fix(arcgis-rest-places): update pagination values for latest API #1109

Closed patrickarlt closed 1 year ago

patrickarlt commented 1 year ago

The places API will change the response structure for pagination info soon. This fixes REST JS to support both the old and new style responses so we can do a release an have everything ready to go when the change happens.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1109 (141fa6e) into main (eca1c03) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 141fa6e differs from pull request most recent head d41e50d. Consider uploading reports for the commit d41e50d to get more accurate results

@@            Coverage Diff            @@
##              main     #1109   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          163       163           
  Lines         3003      3003           
  Branches       534       534           
=========================================
  Hits          3003      3003
Impacted Files Coverage Δ
...es/arcgis-rest-places/test/mocks/nearPoint.mock.ts 100.00% <ø> (ø)
...arcgis-rest-places/test/mocks/withinExtent.mock.ts 100.00% <ø> (ø)
packages/arcgis-rest-places/src/utils.ts 100.00% <100.00%> (ø)