Skip to content
This repository has been archived by the owner on Nov 6, 2024. It is now read-only.

General updates and added a new Known Limitations section #297

Merged
merged 2 commits into from
Aug 24, 2022

Conversation

andygup
Copy link
Member

@andygup andygup commented Aug 23, 2022

This PR includes:

  • General wording cleanup and updated various links.
  • Added a new Known Limitations sections.
  • Added a Known Limitation note about requiring async/await support at runtime for the APIs AMD modules in a future release. This will affect Angular apps using esri-loader.
  • Removed several links and sections related to old SDK guide topics about frameworks.

@andygup andygup requested review from odoe and tomwayson August 23, 2022 18:03
@andygup andygup self-assigned this Aug 23, 2022
Copy link
Collaborator

@odoe odoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

README.md Outdated

<a id="why-is-this-needed"></a>Unfortunately, prior to version 4.18, you couldn't simply `npm install` the ArcGIS API and `import` its modules. The only reliable way to load ArcGIS API for JavaScript modules was using Dojo's AMD loader. Rather than using Dojo to build your application, esri-loader provides a way to dynamically load modules at runtime from a hosted build of the ArcGIS API into applications built using modern tools and framework conventions. This allows your application to take advantage of the fast cached [CDN](https://developers.arcgis.com/javascript/latest/guide/get-api/#cdn).
<a id="why-is-this-needed"></a>esri-loader provides a way to dynamically load the API's AMD modules at runtime from the [ArcGIS CDN](https://developers.arcgis.com/javascript/latest/install-and-set-up/#amd-modules-via-arcgis-cdn) into applications built using modern tools and framework conventions. This allows your application to take advantage of the fast cached [CDN](https://developers.arcgis.com/javascript/latest/guide/get-api/#cdn).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to be clear about the fact that if you are using a version that's before 4.18, you need esri-loader.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes pushed.

@tomwayson tomwayson merged commit 09e330c into master Aug 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants