Cannot use with r.js compiler of requirejs AMD system

[nicksyndicate]

Hi there!

I cant compile in one file my amd modules (with you dependencies) because here https://github.com/amplitude/Amplitude-Javascript/blob/master/amplitude.js#L5 writen this code

return require('1')

can you change it on correct AMD wrapper
Smth like this

(function (root, factory) {
  if(typeof define === "function" && define.amd) {
    // Now we're wrapping the factory and assigning the return
    // value to the root (window) and returning it as well to
    // the AMD loader.
    define(["postal"], function(postal){
      return (root.myModule = factory(postal));
    });
  } else if(typeof module === "object" && module.exports) {
    // I've not encountered a need for this yet, since I haven't
    // run into a scenario where plain modules depend on CommonJS
    // *and* I happen to be loading in a CJS browser environment
    // but I'm including it for the sake of being thorough
    module.exports = (root.myModule = factory(require("postal")));
  } else {
    root.myModule = factory(root.postal);
  }
}(this, function(postal) {
  // module code here....
  return myModule;
}));

Thank you!

[ravangen]

I also have the same root problem with return require('1') that I've been trying to work around in the past couples days. So far I haven't found a solution while still being async. Would love to see this fixed.

[iambumblehead]

+1 for this --please use standard umd so that we can use your script without having to edit it

https://github.com/umdjs/umd/blob/master/templates/returnExportsGlobal.js

[ajerez]

same problem here.
@iambumblehead Do you have a working copy of amplitude with the standard umd wrapper? Thanks

[ravangen]

You can load the script synchronously or edit the script (which you would need to do for each version upgrade).

[blazzy]

Hey Folks, the latest version v3.6.0 is now being built as a proper umd module. Hopefully that solves the problems you were having.