Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

clarify output of integrate command #161

Closed
mykmelez opened this issue Nov 10, 2015 · 0 comments
Closed

clarify output of integrate command #161

mykmelez opened this issue Nov 10, 2015 · 0 comments
Assignees
Milestone

Comments

@mykmelez
Copy link
Contributor

The integrate command's output should be clearer. Here's an example of the current output:

[16:03:05] Your app needs to load the script offline-manager.js in order to register the service
worker that offlines your app. To load the script, add this line to your app's HTML
page(s)/template(s):

    <script src="offline-manager.js"></script>

Here's a proposed redesign:

Integrating Oghliner into the app in the current directory…

✓ Copying offline-manager.js to [TARGET DIRECTORY]… done!

Oghliner has been integrated into the app!

The app needs to load the script offline-manager.js in order to register
the service worker that offlines the app. To load the script, add this line
to the app's HTML page(s)/template(s):

<script src="offline-manager.js"></script>

And commit the changes and push the commit to the origin/master branch:

git commit -m"integrate Oghliner" --all
git push origin master

Then you can offline and deploy the app using the *offline* and *deploy* commands.

ℹ For more information about offlining and deployment, see:
    https://mozilla.github.io/oghliner/

As with #148 and #160, the checkmark is green, the information message (ℹ) is blue, and the Git command examples are bold gray. The HTML tag to copy/paste should also be bold gray. Unsure about indentation level for the HTML tag. Let's left-align for now and see how it feels.

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

No branches or pull requests

2 participants