Titanium-CommonJS icon indicating copy to clipboard operation
Titanium-CommonJS copied to clipboard

Found typo while browsing...

Open Mantic opened this issue 14 years ago • 3 comments

line 67 of lib/worker.js:

Titanium.API.debugh("create Worker()");

is that supposed to be "debugh" vs "debug"?

Mantic avatar Nov 26 '11 22:11 Mantic

Yeah, i'm sure it is. I believe they have require() built into titanium now, so i haven't checked on it lately.

dmachi avatar Nov 27 '11 00:11 dmachi

They have one on their mobile side of things, but I still haven't seen it as of the 1.2.0 desktop SDK. I got your module to install and run, but no matter what it returns with an error: "Unable to resolve require: js/test". (given that I passed 'js/test' to the function) I suppose that's another issue though...

Mantic avatar Nov 27 '11 00:11 Mantic

Yeah, it is probably way out of date. I'd recommend using the latest dojo and load it with the new loader (not in backcompat mode) Since the newer versions of dojo are basically following AMD, it is not as critical to load the way it did before...or even better replace this module by building the loader in as a module. There is a dojo dev that works on Titanium, I can ping him and ask him what he does for this currently if you like.

Dustin

On Nov 26, 2011, at 7:50 PM, Michael Whitaker wrote:

They have one on their mobile side of things, but I still haven't seen it as of the 1.2.0 desktop SDK. I got your module to install and run, but no matter what it returns with an error: "Unable to resolve require: js/test". (given that I passed 'js/test' to the function) I suppose that's another issue though...


Reply to this email directly or view it on GitHub: https://github.com/dmachi/Titanium-CommonJS/issues/1#issuecomment-2885035

dmachi avatar Nov 27 '11 03:11 dmachi