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

<r:page:published_at /> needs a default value #54

Closed
architarious opened this issue Aug 28, 2017 · 1 comment
Closed

<r:page:published_at /> needs a default value #54

architarious opened this issue Aug 28, 2017 · 1 comment

Comments

@architarious
Copy link

<r:page:published_at /> doesn't appear to have a default/fallback value like <r:article:published_at /> does. If you're using conditional logic to parse anything by published date, it will break unless the user remembers to create a published_at: field within their mock-data file.

Examples for current mock-data workarounds:

  • published_at: <%= Time.now %>
    OR
  • published_at: "2017-08-14 13:47:19 -0400"
@zeroedin
Copy link
Contributor

So this commit fix is currently in the test branch. This branch also brings vagrant and virtual box up to current versions. Will be testing this branch for a bit then moving it to dev. Then finally merging with master after a detailed Changelog is created. There were some modifications made in dev that changes things about mock_data defaults that people have to be made aware of.

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

No branches or pull requests

2 participants