Skip to content

[patch] fixing update method so that it takes into account a string for the a…#1360

Merged
particlebanana merged 2 commits intomasterfrom
unknown repository
Jun 8, 2016
Merged

[patch] fixing update method so that it takes into account a string for the a…#1360
particlebanana merged 2 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented May 28, 2016

…utoUpdatedAt attribute (was not updating the database record with the current timestamp)

…utoUpdatedAt attribute (was not updating the database record with the current timestamp)
@sailsbot
Copy link
Copy Markdown

Hi @jenjenut233! It looks like your pull request title doesn’t quite conform to our guidelines. Please edit the title so that it starts with [proposal], [patch], [fixes #], or [implements #]. Once you've fixed it, post a comment below (e.g. "ok, fixed!") and we'll take a look!

@ghost ghost changed the title fixing update method so that it takes into account a string for the a… [patch] fixing update method so that it takes into account a string for the a… May 28, 2016
@ghost
Copy link
Copy Markdown
Author

ghost commented May 28, 2016

fixed

@joshuamarquez
Copy link
Copy Markdown

same problem here

@particlebanana
Copy link
Copy Markdown
Contributor

Thanks @jenjenut233!

@particlebanana
Copy link
Copy Markdown
Contributor

particlebanana commented Jun 8, 2016

Published Waterline 0.11.3 to npm (for use with Sails) and will publish a new version of 0.12 in a bit.

dynamiccast added a commit to dynamiccast/sails-json-api-blueprints that referenced this pull request Jun 23, 2016
… names

Because of a bug in Waterline prior to version 0.11.3, updatedAt attributes needs to be handle manually during PATCHes
See balderdashy/waterline#1360 for more details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants