Gavin Rehkemper

Results 76 comments of Gavin Rehkemper

Fixed in [v3.0.12](https://github.com/Esri/esri-leaflet/releases/tag/v3.0.12).

Thanks @pmacMaps! We are working on updating the warning language [here](https://developers.arcgis.com/esri-leaflet/api-reference/layers/tiled-map-layer/) and updating the non-mercator projection samples. The changes should be reflected on the documentation site soon.

This is happening fairly regularly on the automated pull request tests: https://github.com/Esri/arcgis-rest-js/actions/runs/9199343301/job/25303924894?pr=1159 ![image](https://github.com/Esri/arcgis-rest-js/assets/137905994/b040887b-8e90-4b1b-974f-0bca52b1129c)

Hi Arjan, thanks for logging this. This snippet is actually old - using the JS API v3 syntx (for example, should be `esri/identity/IdentityManager` instead of `esri/id`). Could you please send...

## Snippet (non-)issue After further looking into this, I think [that snippet](https://developers.arcgis.com/arcgis-rest-js/api-reference/arcgis-rest-request/ArcGISIdentityManager/#fromCredential) is actually acceptable: - [The snippet](https://developers.arcgis.com/arcgis-rest-js/api-reference/arcgis-rest-request/ArcGISIdentityManager/#fromCredential) uses [findCredential](https://developers.arcgis.com/javascript/latest/api-reference/esri-identity-IdentityManager.html#findCredential), which "_Returns the [Credential](https://developers.arcgis.com/javascript/latest/api-reference/esri-identity-Credential.html) for the resource identified by the...

Discussed with @patrickarlt, we propose to **remove** this line: https://github.com/Esri/arcgis-rest-js/blob/7d8220ad3d3a872fda36f02c21e4046a2b87b7ed/packages/arcgis-rest-request/src/ArcGISIdentityManager.ts#L97