Skip to content

Add ajax library#344

Merged
rvagg merged 1 commit intomicrojs:masterfrom
ForbesLindesay:patch-1
Dec 9, 2012
Merged

Add ajax library#344
rvagg merged 1 commit intomicrojs:masterfrom
ForbesLindesay:patch-1

Conversation

@ForbesLindesay
Copy link
Copy Markdown
Contributor

Add ajax

@rvagg
Copy link
Copy Markdown
Contributor

rvagg commented Nov 29, 2012

Sorry @ForbesLindesay, I'm going to have to reject this for now because you have a dependency, component/type, and no-deps is one of the limitations of listing on microjs.com. Feel free to submit another PR if you end up removing that dependency at some point. Also it looks like a pretty useful lib outside of a component build, perhaps you could make it work fully stand-alone as well?
Cheers.

@rvagg rvagg closed this Nov 29, 2012
@ForbesLindesay
Copy link
Copy Markdown
Contributor Author

The source linked to is a standalone dependency-free build. The build process automatically includes the dependent in a wrapper and it's still within the size limit. To use that you literally just add it in a script tag and use the global ajax to reference it, with no dependencies. Component is really just the build process.

@rvagg
Copy link
Copy Markdown
Contributor

rvagg commented Nov 29, 2012

arrr.. I should have looked at your source link, sorry about that.

@rvagg rvagg reopened this Nov 29, 2012
@rvagg
Copy link
Copy Markdown
Contributor

rvagg commented Dec 9, 2012

👍

@rvagg rvagg merged commit f135303 into microjs:master Dec 9, 2012
@ForbesLindesay ForbesLindesay deleted the patch-1 branch December 24, 2012 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants