Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

Add models directory for new generators #162

Conversation

DesignByOnyx
Copy link
Contributor

@DesignByOnyx DesignByOnyx commented Feb 1, 2017

This fixes #94 for the new generators. This separates models into their own directory and allows sequelize models to set up relationships which are sync'd to the database.


app.set('sequelizeClient', sequelize);

app.setup = function(...args) {
// Set up data relationships
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would call const result = oldSetup.apply(this, args); first and then return result; in the end just in case.

@daffl
Copy link
Member

daffl commented Feb 1, 2017

Just one comment, then I'll merge it in. Thanks again, this is great!

@daffl daffl merged commit 909983b into feathersjs-ecosystem:generator-refactoring Feb 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants