-
Notifications
You must be signed in to change notification settings - Fork 226
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
wicket-arcgis AMD update #74
Comments
Lol, I commented on your PR without reading this first. Now it makes a lot more sense. Could you provide an example of the provider reccomended way of loading esri components? |
From the docs. require(["esri/map"], function(Map) { /* code goes here */ }); You would reference |
I'm not sure why the Dojo AMD loader isn't able to load (function(root, factory) {
if (typeof define === "function" && define.amd) {
// AMD (+ global for extensions)
define(function() {
return factory();
}); I've tried to work with the Dojo require(['esri/map', 'https://arthur-e.github.io/Wicket/wicket.js'], function (Map) { However, what we need is this, which does not work: require(['esri/map', 'wicket.js'], function (Map) { But, again, since the |
this component only works with the global esri namespace instead of the AMD modules which is the not recommended route for working with esri js.
My take is that it doesn't really fit nicely into your project because you are not on the AMD train and new objects are being created from the global namespace. In AMD land you would need to require that class in to create instances of it. It could come in as a parameter or a property of your global though.
What are your thoughts on this?
The text was updated successfully, but these errors were encountered: