search

feathersjs-ecosystem / generator-feathers

A Yeoman generator for a Feathers application
https://github.com/feathersjs/feathers
MIT License
120 stars 132 forks source link

knex table name should snake_case #288

Closed nueko closed 6 years ago

nueko commented 6 years ago

Hi, the generated model for knex use kebab-case. it should be snake_case.

daffl commented 6 years ago

A PR changing kebabName in https://github.com/feathersjs/generator-feathers/blob/master/generators/service/templates/model/knex.js and https://github.com/feathersjs/generator-feathers/blob/master/generators/service/templates/model/knex-user.js to snakeName would be very welcome 😄

nueko commented 6 years ago

done #289 but forgot to add fix keyword

daffl commented 6 years ago

Closed via #289