Skip to content
This repository has been archived by the owner on Sep 8, 2020. It is now read-only.

Locally runnable #30

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Locally runnable #30

wants to merge 4 commits into from

Conversation

motin
Copy link

@motin motin commented Oct 20, 2015

I expected to be able to run and try out the modules. Did not work out. This PR was as long as I got before I realized that the project was never meant to be run locally.

It would however be a great thing to be able to do. A mock Authenticated/Authentication module seems to be missing and an api blueprint that can be used with npm-mock or similar to set up a local mock REST api server? Something like https://github.com/neam/auth0-mock-api?

@ProLoser
Copy link
Member

Keep in mind there's some mild disagreement about my styleguide from some other ui guys and I'd really like them to chime in with their 2c.

Anyway, does this necessarily need to be in a client folder when the entire codebase could theoretically be the client folder? That way you can have a standalone client and a standalone server. I realize they are normally developed hand in hand it's just a thought.

@motin
Copy link
Author

motin commented Oct 21, 2015

Anyway, does this necessarily need to be in a client folder when the entire codebase could theoretically be the client folder? That way you can have a standalone client and a standalone server. I realize they are normally developed hand in hand it's just a thought.

I just moved the files according to the paths already specified in the Gulpfile. I could to either that or change the paths in the Gulpfile, but it seemed awkward to have the destination path of the build inside the source path, so I went with the original paths instead.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants