Front-matter `permalink` override not working about permalinks HOT 7 CLOSED

As far as I can tell, this bug is still valid, and should be reopened. @Ajedi32, the code you linked to seems to be on a different fork. Unless I'm missing something, that code isn't in master, at the moment.

from permalinks.

That code is on master: https://github.com/leolabs/metalsmith-permalinks/blob/master/lib/index.js#L81-L83

Are you using the version from master? Or are you using the latest stable release?

from permalinks.

The permalink front-matter is a boolean value, eg...

---
title: "Hello Metalsmiths"
permalink: true
---

The URL is determined by the filename, just like all other pages.

Please close this issue.

from permalinks.

@Ajedi32 - The code does seem to be in master. My mistake and sorry for the confusion.

Closing issue. I'll evaluate my local env issues.

@doublejosh The readme.md example doesn't show it as a Boolean:
https://github.com/segmentio/metalsmith-permalinks/#overriding-the-permalink-for-a-file

from permalinks.

Huh, you're right. The docs do show it as both boolean (using the false case to exclude) AND a string to explicitly name the desired path. I had only used the boolean case.

Good luck, please let us know what you find.

from permalinks.

It definitely was on master. Here's the commit: d5f5070 Merged with PR #65.

That code seems to have been removed since then though, in deaa8b9.

from permalinks.

Root cause seems to be that the code was removed when #69 was rebased, and nobody noticed because there were no tests for that feature. To prevent this from happening again, any future fixes for this issue should include tests.

from permalinks.