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

Permalink attribute adds erroneous <p> tags to path #93

Closed
philhawksworth opened this issue Mar 30, 2018 · 3 comments
Closed

Permalink attribute adds erroneous <p> tags to path #93

philhawksworth opened this issue Mar 30, 2018 · 3 comments
Assignees
Labels
Milestone

Comments

@philhawksworth
Copy link

When specifying the permalink for a file via either the front matter or via the folder json config, the path generated is wrapped in an undesired <p> tag.

---
title: "My test file"
permalink: "/news/{{ title | slug }}/index.html"
---

Yielding the following log output when the build runs (where src/sites and dist are my input and output directories):

writing dist/<p>/news/my-test-file/index.html</p> from ./src/site/news/my-test-file.md.

This also happens if the nunjucks templating expression is omitted and a string is used on its own like so:

---
title: "My test file"
permalink: "/news/my-test-file/index.html"
---
  • Eleventy version: 0.3.0
  • my other config related to templating:
    templateFormats : ["njk", "md"],
    htmlTemplateEngine : "njk",
    markdownTemplateEngine : "njk"
@zachleat zachleat self-assigned this Mar 31, 2018
@zachleat zachleat added the bug label Mar 31, 2018
@zachleat
Copy link
Member

@philhawksworth Any templateEngineOverride’s in play here?

@zachleat
Copy link
Member

zachleat commented Apr 1, 2018

I can reproduce this with templateEngineOverride and given our previous discussions gonna assume the answer is yes

@zachleat zachleat added this to the v0.3.1 milestone Apr 1, 2018
@philhawksworth
Copy link
Author

Yes there are templateEngineOverrides in play:

"templateEngineOverride": "njk,md"

Wilto pushed a commit to Wilto/eleventy that referenced this issue Apr 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants