Skip to content

update models - #432

Open
lightmild wants to merge 1 commit into
danliris:devfrom
lightmild:dev
Open

update models#432
lightmild wants to merge 1 commit into
danliris:devfrom
lightmild:dev

Conversation

@lightmild

Copy link
Copy Markdown

update models

@@ -0,0 +1,17 @@
'use strict';

var BaseModel = require('model-toolkit').BaseModel;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'require' is not defined.


var BaseModel = require('model-toolkit').BaseModel;

module.exports = class WorkingHoursStandard extends BaseModel {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'class' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
'module' is not defined.

@@ -0,0 +1,17 @@
'use strict';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use the function form of "use strict".

data.should.have.property('items');
data.items.should.instanceof(Array);

for (var item of data.items) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'for of' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

'use strict';
var BaseModel = require('model-toolkit').BaseModel;
var Unit = require('../master/unit');
var Week = require('./weekly-plan-item');

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'require' is not defined.

var BaseModel = require('model-toolkit').BaseModel;


module.exports = class Style extends BaseModel {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'class' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
'module' is not defined.

@@ -0,0 +1,14 @@
'use strict';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use the function form of "use strict".

@@ -0,0 +1,21 @@
'use strict';
var BaseModel = require('model-toolkit').BaseModel;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'require' is not defined.

'use strict';
var BaseModel = require('model-toolkit').BaseModel;

module.exports = class WeeklyPlanItem extends BaseModel {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'class' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
'module' is not defined.

@@ -0,0 +1,21 @@
'use strict';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use the function form of "use strict".

update models
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants