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

Improvement/ember cli mirage #42

Merged

Conversation

ultimatemonty
Copy link
Member

@gschorkopf @pdougall1 this sets up some super basic mirage routes to enable devs to at least get the site loading locally without the Rails API running. Lots still to be done but this is a start

@@ -50,6 +50,13 @@
</a>
{{/if}}
{{link-to "Frolfr" "index" id="title" classNames="navbar-brand navbar-brand-text navbar-left navbar-link"}}
{{#if currentUser.isAuthenticated}}
Copy link
Member

Choose a reason for hiding this comment

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

Why was this commit included in the PR?

@gschorkopf
Copy link
Member

Looks good!

@gschorkopf
Copy link
Member

I figured we'd run into some merge conflicts after #41

@ultimatemonty ultimatemonty force-pushed the improvement/ember-cli-mirage branch from 26c173a to d432273 Compare August 8, 2016 01:01
@ultimatemonty
Copy link
Member Author

@gschorkopf removed the template commit and the ember-cli-build commit, and merge the upstream master. Should be good to go now

@gschorkopf
Copy link
Member

Seems legit! Thanks for getting this started!

@gschorkopf gschorkopf merged commit 6a74958 into frolfr:master Aug 8, 2016
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.

3 participants