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

Commit

Permalink
Addressing PR Feedback
Browse files Browse the repository at this point in the history
Noted that 'bundle' is optional and has a default value
Set 'bundle' to null if a value is not given
  • Loading branch information
ellenmobify committed Jul 14, 2017
1 parent 7a5168c commit 2138d48
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ The `preview` command uses http://preview.mobify.com to open a website specified
Parameter Name | Parameter Type | Description
------------- | -------------- | -----------
url | String | The URL to preview, equivalent to the Site URL field.
bundle | String | The bundle URL, equivalent to the Bundle Location field.
bundle | String | _optional_ The bundle URL, equivalent to the Bundle Location field. Default is `https://localhost:8443/loader.js`
callback | Function | _optional_ A function to call after the current command finishes execution.

```
Expand Down
2 changes: 1 addition & 1 deletion commands/preview.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ exports.command = function(url, bundle, callback) {

if (typeof bundle === 'function') {
callback = bundle;
bundle = '';
bundle = null;
}

var bundleUrl = bundle || 'https://localhost:8443/loader.js';
Expand Down

0 comments on commit 2138d48

Please sign in to comment.