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

Jump to bottom

standalone Eve apps #571

Merged
merged 30 commits into from Nov 30, 2016
Merged

standalone Eve apps #571

merged 30 commits into from Nov 30, 2016

Conversation

ibdknox
Copy link
Contributor

@ibdknox ibdknox commented Nov 17, 2016 •
edited

This PR includes the changes necessary to allow for standalone Eve apps:

  1. Running eve with a file as an argument will cause Eve to run without loading the editor.
  2. Static file serving for the assets/ directory relative to where eve is run
  3. Ensuring a standalone app can't call into scary editor methods like eval
  4. Making the server database per http connection, allowing your program to run per http request and not worry about what other requests might be in flight

With 4 it's now possible to write json apis and such in Eve as each new http request spins up an Eve evaluation that you can optionally choose to handle. If you don't, index.html is sent and the normal Eve flow kicks off.

@ibdknox ibdknox mentioned this pull request Nov 18, 2016
Closed
@cmontella cmontella mentioned this pull request Nov 18, 2016
Open
@dwsmorris dwsmorris mentioned this pull request Nov 22, 2016
Open
joshuafcole and others added 21 commits November 22, 2016 16:13
@joshuafcole
Pull config out into module, add npmignore, update build phase for pa…
2fa8531 
…ckaging + npm3, fix various url-related bugs
@joshuafcole
Remove workspaces rewrite from server (doesnt work when installed -g)
db9d0ad
@cmontella
Update readme
074db56
@joshuafcole
Fix package-workspaces script to include eve src workspace
6cf8fb5
@joshuafcole
Add eveSource.find to resolve a workspace-relative URL to a file
7d4429f
@joshuafcole
Fix fully local-initialization
1b657c1
@joshuafcole
Merge branch 'npm-package' of github.com:/witheve/eve into npm-package
d521b45
@joshuafcole
Add fixme about file arg in eve binary
8b8b96b
@joshuafcole
Fix crash when trying to load nonexistent document
76c29fd
@joshuafcole
Fix crash when loading nonexistent document, add getRelativePath, don…
1d34dfb 
…t traverse into node_modules when finding workspace files
@joshuafcole
Only attempt to load sane documents
03ab7c3
@joshuafcole
Dont explode when initializing without code
0679a34
@joshuafcole
Fix editor workspace/document loading in external projects
58463ac
@joshuafcole
Fix port flag not overriding ENV and file argument following a boolea…
bd67f07 
…n flag
@joshuafcole
Remove console logs
2d08d27
@joshuafcole
Remove vestigial code
e3aaa1a
@joshuafcole
Standardize variable names
a24ecae
@joshuafcole
Move document saving into eveSource and separate runtimeOwner and con…
10f0dc1 
…trolOwner, fixing saving external documents to disk
@joshuafcole
Fix loading quickstart on global install + local control when running…
c24d114 
… examples globally
@joshuafcole
Fix broken path logic
deb8277
@joshuafcole
Dont explode on non-existent documents
0b00981
joshuafcole
joshuafcole approved these changes Nov 30, 2016
View reviewed changes
Copy link
Contributor

@joshuafcole joshuafcole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍶 ♐ 🌚

@ibdknox ibdknox merged commit d368c32 into master Nov 30, 2016
@ibdknox ibdknox deleted the app-loading branch November 30, 2016 00:58
@joshuafcole joshuafcole mentioned this pull request Dec 6, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joshuafcole joshuafcole joshuafcole approved these changes

Assignees

@joshuafcole joshuafcole

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ibdknox @joshuafcole @cmontella