search

donejs / generator-donejs

A Yeoman generator for your DoneJS application
MIT License
9 stars 7 forks source link

Generator should follow the debugging guide by include include the name/tag as the 0th argument to extended DefineMap (the view model) #283

Open mjstahl opened 6 years ago

mjstahl commented 6 years ago

The debugging guide recommends naming your DefineMaps by putting a string in the 0th argument of the .extend.

The app generator includes a conditional import for can-debug. This implies that we should be using the new debugging features.

But when I generate a new component, the view model does not include a name for the DefineMap as recommended by the guide.

Per a discussion on slack, we should camelcase the tag (after removing the -) and maybe add VM at the end. (like so - https://github.com/canjs/can-component/blob/311a68be00f9d7a5ce74f2e68f75ddfbffb1ffef/can-component.js#L175)