Add permalink to include the page #40

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@npeters

npeters commented Jan 17, 2014

permalink attribute can be use to include a page like a post

Add permalink to include the page
permalink attribute can be use to include a page like a post
@@ -251,9 +251,7 @@ module.exports = function(params, callback) {
if(_.isUndefined(options.structure) && _.isUndefined(options.preset)) {
page.dest = page.dest;
} else {
- if (page.basename === 'index') {

This comment has been minimized.

Show comment Hide comment
@jonschlinkert

jonschlinkert Jan 17, 2014

Owner

this code is necessary to exclude index pages from having the destination file path altered in an unexpected way.

@jonschlinkert

jonschlinkert Jan 17, 2014

Owner

this code is necessary to exclude index pages from having the destination file path altered in an unexpected way.

- if (page.basename === 'index') {
- page.dest = page.dest;
- } else {
+ if (page.data.permalink === true ) {

This comment has been minimized.

Show comment Hide comment
@jonschlinkert

jonschlinkert Jan 17, 2014

Owner

how does this achieve what you're suggesting in the pr?

@jonschlinkert

jonschlinkert Jan 17, 2014

Owner

how does this achieve what you're suggesting in the pr?

@jonschlinkert

This comment has been minimized.

Show comment Hide comment
@jonschlinkert

jonschlinkert Jul 15, 2015

Owner

closing based on age and lack of response

Owner

jonschlinkert commented Jul 15, 2015

closing based on age and lack of response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment