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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Renamed project to Marvin, updated readme #20

Merged
merged 1 commit into from May 22, 2017

Conversation

Projects
None yet
3 participants
@Stanko
Member

Stanko commented May 20, 2017

This ended up being bigger update than I initaly planned. Anyway, I renamed project to Marvin, added Work & Co logo to example page, along with much nicer CSS.

Feel free to correct my english 馃檭

Once we merge this one, I'll rename repo to marvin.

EDIT: It is easier to check whole readme file here

@Stanko Stanko requested review from oliverdore-work and nemanjawork May 20, 2017

@nemanjawork

nemanjawork approved these changes May 20, 2017 edited

Great stuff on the naming, I like the idea!

The PR looks good to me, let's 馃殺

@@ -174,6 +174,7 @@ module.exports = {
inline: !isProduction,
hot: !isProduction,
host: '0.0.0.0',
disableHostCheck: true,

This comment has been minimized.

@Stanko

Stanko May 21, 2017

Member

To allow testing from local network.

@Stanko

Stanko May 21, 2017

Member

To allow testing from local network.

@oliverdore-work

Looks great. Thanks for the updates.

@Stanko Stanko merged commit ac35431 into master May 22, 2017

@Stanko Stanko deleted the marvin branch May 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment