search

googlearchive / generator-angularfire

Yeoman generator for AngularFire: Angular + Firebase apps
191 stars 52 forks source link

Error "btnDanger" is not defined when not including Bootstrap #19

Closed mikemcbride closed 9 years ago

mikemcbride commented 9 years ago

Steps: I entered my firebase instance I opted to include FirebaseSimpleAuth (email/password and anonymous) I declined Sass I declined Bootstrap I accepted all Angular modules

Error: undefined:38 ((t = ( btnDanger )) == null ? '' : t) + ^ ReferenceError: btnDanger is not defined at eval (eval at template (/usr/local/lib/node_modules/generator-angularfire/node_modules/yeoman-generator/node_modules/lodash/dist/lodash.js:6305:22), :38:11) at Generator.underscore as _engine at Generator.engine (/usr/local/lib/node_modules/generator-angularfire/node_modules/yeoman-generator/lib/actions/actions.js:303:10) at Generator.template (/usr/local/lib/node_modules/generator-angularfire/node_modules/yeoman-generator/lib/actions/actions.js:281:15) at Generator._htmlTpl (/usr/local/lib/node_modules/generator-angularfire/app/index.js:490:8) at Generator.copyAngularFireFiles (/usr/local/lib/node_modules/generator-angularfire/app/index.js:418:10) at /usr/local/lib/node_modules/generator-angularfire/node_modules/yeoman-generator/lib/base.js:341:43 at /usr/local/lib/node_modules/generator-angularfire/node_modules/yeoman-generator/node_modules/async/lib/async.js:551:21 at /usr/local/lib/node_modules/generator-angularfire/node_modules/yeoman-generator/node_modules/async/lib/async.js:227:13 at iterate (/usr/local/lib/node_modules/generator-angularfire/node_modules/yeoman-generator/node_modules/async/lib/async.js:134:13)

katowulf commented 9 years ago

Already corrected and merged; https://github.com/firebase/generator-angularfire/pull/16

somecallmejosh commented 9 years ago

Hi there, just a heads up: I just updated this generator and am still getting the same error if I include bootstrap.

mikemcbride commented 9 years ago

@somecallmejosh I'd add this comment on #16 instead of here for more visibility since that's where the fix was documented, or open a new issue. Marking this one closed to avoid confusion going forward

somecallmejosh commented 9 years ago

Doh! Thanks, Mike.

mikemcbride commented 9 years ago

No worries! :smile: