Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New FCC Models, Service, Controller, Module and Class layers #80

Merged
merged 7 commits into from
Jun 7, 2014

Conversation

pilsy
Copy link
Contributor

@pilsy pilsy commented Jun 7, 2014

  • New First Class Citizen Models allow a single definition of a model that works with both ORM and ODM modules
  • New Service Layer provided from the Classes util, to work with FCC Models, API has been standardised for use with both ORM and ODM modules
  • Removed BaseService
  • Controller has been modified to use a newer version of clever-controller that allows ObjectId's to be used as id's in routes.
  • Controller has been updated to work with the new Service Layer.
  • Module Layer has been changed, you now need to return Module.extend() from your module instead of returning a new instance that you created inside module//module.js
  • All Module classes now have a this.debug() helper to simplify the debugging process, to debug any module simply set environment variable DEBUG to the camelcased name of the module (clever-orm = cleverOrm)
  • Modules now have their package.json file loaded and exposed as Module.pkg
  • Hooks have been reorganised and optimised for all changes
  • Modified the Module Loader to work with the new Module Layer
  • New DEBUG environment variable names, DEBUG=Services,Models,Modules,ModuleLoader
  • New injector magic module, require( 'injector' )

pilsy added a commit that referenced this pull request Jun 7, 2014
New FCC Models, Service, Controller, Module and Class layers
@pilsy pilsy merged commit 822e6d6 into master Jun 7, 2014
@pilsy pilsy deleted the feature/modules-in-injector branch June 17, 2014 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant