Skip to content

Commit

Permalink
Merge pull request #33 from isleofcode/fix/cordovaid
Browse files Browse the repository at this point in the history
fix(cordovaid): use lowercased `cordovaid` since cordova doesn't like…
  • Loading branch information
anulman committed Apr 27, 2016
2 parents 704a0ae + b5a9fd0 commit 524a129
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ If you are migrating from ember-cli-cordova, as a pre step move your
ember install ember-cordova
```

You can optionally pass name & cordovaId params, which will set the id &
You can optionally pass name & cordovaid params, which will set the id &
display name of your Cordova application. If none exist, your Ember Apps
name will be used.

Expand Down
4 changes: 2 additions & 2 deletions blueprints/ember-cordova/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ module.exports = {
name: 'name',
type: String
}, {
name: 'cordovaId',
name: 'cordovaid',
type: String
}
],
Expand All @@ -33,7 +33,7 @@ module.exports = {
var projectName = this.project.name();

var create = new CreateTask({
id: options.cordovaId || camelize(projectName),
id: options.cordovaid || camelize(projectName),
name: options.name || camelize(projectName),
project: this.project,
ui: this.ui
Expand Down

0 comments on commit 524a129

Please sign in to comment.