search

simonedelmann / crud-kit

CRUD for Vapor 4. We all write CRUD (Create-Read-Update-Delete) routes all the time. The intention of this package is to reduce repeating code and to provide a fast start for an API.
MIT License
56 stars 8 forks source link

Crud vs CRUD #1

Closed kaishin closed 4 years ago

kaishin commented 4 years ago

Hi Simon,

Kudos for the work! I think this is going to be very useful for quickly bootstrapping projects. I have some feedback about naming conventions in the API, however.

CRUD being an abbreviation, it looks a bit odd to spell it as Crud. If it's meant as a pun, then it's unfortunate that crud in english has only negative connotations. I also think that the protocol name Crudable is, for lack of a better word, off-putting.

What do you think of the following?

import CRUDKit
extension Todo: CRUDModel { }
simonedelmann commented 4 years ago

Hey, thanks for your feedback! As English is not my first language, naming things is always not easy for me. I will change this right now!

simonedelmann commented 4 years ago

...and done! Thanks again for your suggestion!

kaishin commented 4 years ago

Very glad you considered my suggestion. Looking forward to using this soon!