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

Already on GitHub? Sign in to your account

added presence validation for published_at and author #272

Merged
merged 2 commits into from Sep 5, 2012

Conversation

Projects
None yet
3 participants
Contributor

keram commented Aug 20, 2012

hello,
these attributes are required thus I added presence validation
this also validate user_id against valid exists user

This pull request fails (merged c9734fb into 50661cc).

Owner

parndt commented Aug 20, 2012

Hi thanks - unfortunately this breaks most of the tests as they are now coming up with validation errors. See http://travis-ci.org/#!/resolve/refinerycms-blog/jobs/2177679 for an example. Are you able to take care of this? Probably it just means updating the factories.

Contributor

keram commented Aug 20, 2012

ok, I can do that

This pull request passes (merged 4cfd546 into 50661cc).

Owner

parndt commented Aug 21, 2012

@joemsak look good to you?

@parndt parndt merged commit 4cfd546 into refinery:master Sep 5, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment