Skip to content

Add autoValue for created_at and updated_at#1661

Merged
jykae merged 1 commit intodevelopfrom
feature/fix-api-created-updated-dates
Sep 29, 2016
Merged

Add autoValue for created_at and updated_at#1661
jykae merged 1 commit intodevelopfrom
feature/fix-api-created-updated-dates

Conversation

@brylie
Copy link
Copy Markdown
Contributor

@brylie brylie commented Sep 29, 2016

Closes #1595
Closes #1652

Changes

  • autoValue Api created_at field on insert
  • autoValue Api updated_at field on update

@jykae
Copy link
Copy Markdown
Contributor

jykae commented Sep 29, 2016

@brylie reviewing

}
},
},
created_by: {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@brylie I think we should add value on creation for this, if we want to keep this field and make use of it at some point.

}
},
},
updated_by: {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also value here on update.

@jykae
Copy link
Copy Markdown
Contributor

jykae commented Sep 29, 2016

Decided with @brylie to fix created/updated by fields as enhancement task #1663

@jykae
Copy link
Copy Markdown
Contributor

jykae commented Sep 29, 2016

Merging

@jykae jykae merged commit f853111 into develop Sep 29, 2016
@jykae jykae removed the in progress label Sep 29, 2016
@jykae jykae deleted the feature/fix-api-created-updated-dates branch September 29, 2016 12:41
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.

3 participants