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.

Sign up for GitHub

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

Dummy App #7

Closed
wants to merge 2 commits into from
Closed

Dummy App #7

wants to merge 2 commits into from

Conversation

gr2m
Copy link
Member

@gr2m gr2m commented Jul 13, 2016 •
edited
Loading

WORK IN PROGRESS

in preparation for #5

@gr2m
Copy link
Member Author

gr2m commented Jul 14, 2016

So it seems to work in general, but no CSS gets loaded, any idea why @courajs

@gr2m
Copy link
Member Author

gr2m commented Jul 14, 2016 •
edited
Loading

I’m trying to setup acceptance test, but looks like it cannot connect to Hoodie. Might it be something with the ports? I actually don’t even understand why ember server is starting the Hoodie server? wouldn’t we need to add the logic to server/index.js? Or is ember server requiring the index.js file and running the serverMiddleware?

@courajs
Copy link
Contributor

courajs commented Jul 18, 2016

I think server/index and serverMiddleware are meant mainly for development, not test. Usually ajax calls are mocked out in test with pretender or similar. I haven't looked into how we would get the server running in test yet

@courajs
Copy link
Contributor

courajs commented Jul 18, 2016

No reason off the top of my head the css wouldn't load, I'd have to dig into it.

@courajs courajs mentioned this pull request Aug 13, 2016
Merged
@courajs
Copy link
Contributor

courajs commented Aug 13, 2016

Closing in favor of #16

@courajs courajs closed this Aug 13, 2016
@gr2m gr2m deleted the 5/dummy-app branch August 13, 2016 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gr2m @courajs