sails-generate-models icon indicating copy to clipboard operation
sails-generate-models copied to clipboard

Published 20 hours ago • verified publisher icon SpoonX

Sails v1 MySQL Data Models

Open jovanross opened this issue 8 years ago • 7 comments
trafficstars

I created this fork in order to manage data models for Sails v1. There are some key differences between the original project.

  1. I've added data types that v1 now supports.

  2. My camel-case handling is a bit different due to how my own project needs were concerned (I have MySQL tables with underscores), so I added and am using a ucwords function instead of using ucfirst .

  3. Sails v1 data models formatting has been updated along with some handling of referential data.

I haven't tested this with Sails versions earlier than v1 so I'm not completely sure of backward compatibility....hence why I haven't submitted these changes earlier. Hopefully someone can find it useful...or a stepping stone to something useful.

Thanks!

jovanross avatar Oct 11 '17 16:10 jovanross

@jovanross Thanks! Just, fyi, I didn't mention you in a negative way. I hope it didn't sound like that.

If this works well for sails v1 (I'm guessing you mean waterline v1?), I think it could be merged in such a way that the version is a choice you can make on the cli, or a version you can install specifically. (I'll have to take your word for it for now, as I don't use waterline anymore). Once merged, I'll tag this up (I'll have to do some branch magic first).

RWOverdijk avatar Oct 12 '17 07:10 RWOverdijk

No, I didn't take anything negatively at all.

I'm not sure if the waterline version is maintained at the same level as sails, but yes, this is for whatever waterline version is compatible with the new sails v1. http://sailsjs.com/documentation/upgrading/to-v-1-0

I'm starting to migrate a large PHP project to sails so I'll probably be active in updating this project over the next year or so at least.

jovanross avatar Oct 12 '17 14:10 jovanross

@jovanross I sent you a message on gitter. Are you active there?

RWOverdijk avatar Oct 13 '17 08:10 RWOverdijk

Any chance for this being merged in the main branch? Running into an issue now with datastores.js. Will try to implement this on my local install for now though. Thanks

risingsunomi avatar May 14 '18 17:05 risingsunomi

@risingsunomi It can be if it works. It'll need to be a major version bump and the readme needs to be updated accordingly. Also, there appears to be a conflict. If you're willing to make those minimal changes I'd be more than happy to merge this.

RWOverdijk avatar May 15 '18 06:05 RWOverdijk

Hi, is it possible to use this changes for Sails v1.0 now?

dzarezenko avatar Jan 29 '19 11:01 dzarezenko

@dzarezenko Sure thing. I'll release updates for PRs and do maintainer stuff but I won't work on the repo anymore.

I work with stixjs and wetland ORM now.

RWOverdijk avatar Jan 29 '19 11:01 RWOverdijk