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

Added hierarchical_pages plugin. #178

Merged
merged 1 commit into from Oct 15, 2016
Merged

Added hierarchical_pages plugin. #178

merged 1 commit into from Oct 15, 2016

Conversation

felixfontein
Copy link
Contributor

This is a demonstration how getnikola/nikola#2502 could be used to translate the full URLs of pages without repeating information (like for example the .. path: xx solution described in getnikola/nikola#2116 (comment)).

MinVersion = 7.8.2

[Documentation]
Author = Roberto Alsina and others
Copy link
Member

@Kwpolska Kwpolska Oct 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it really? (same in © notice)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since most of the code is copy'n'pasted from various Nikola parts, kind of yes.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The typical way i’d solve it would be

Roberto Alsina, Felix Fontein and others

but that’s just my thing. (In any case, you’re the one who should get the blame for any bugs)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also changed.

- `https://example.com/about/team/roberto-alsina/`

Now assume you want to make your homepage available in more languages, say
also in german. You want the URLs for the translated posts to be:
Copy link
Member

@Kwpolska Kwpolska Oct 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

English* above, German* here (and in other places)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@felixfontein felixfontein merged commit e9f67ba into getnikola:master Oct 15, 2016
@felixfontein
Copy link
Contributor Author

Whoops, looks like I (again) pushed to the wrong repo... Sorry!

@getnikola getnikola locked and limited conversation to collaborators Oct 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants