Skip to content
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

Connect middleware and npm package for serving ace from a node.js app. #217

Closed
wants to merge 5 commits into from
Closed

Conversation

aslakhellesoy
Copy link

This small patch would make it extremely easy to get Ace served from a node.js app:

npm install npm-connect

Then serve it up:

server.use(require('ace-connect')('/js/ace'));

Ace is here -> http://localhost:3000/js/ace/ace.js

If you accept this patch there will be two npm packages for ace:

  • ace - the current one
  • ace-connect - a prebuilt one intended to be embedded in a node/connect app
    @Gissues:{"order":77.63975155279513,"status":"backlog"}

@aslakhellesoy
Copy link
Author

Has anyone had a chance to look at this? Is it something you'd consider for the next release?

@lennartcl
Copy link
Contributor

Today we're cleaning up and closing all pull requests that haven't had recent activity and were filed over six months ago. If you'd like to update the request or have comments, please respond or reopen the pull request.

@lennartcl lennartcl closed this Aug 6, 2012
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