Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

User defined REST URI path does not display #27

Closed
ghost opened this issue May 23, 2016 · 4 comments · Fixed by #31
Closed

User defined REST URI path does not display #27

ghost opened this issue May 23, 2016 · 4 comments · Fixed by #31

Comments

@ghost
Copy link

ghost commented May 23, 2016

Rendered HTML shows the text display representation of the relative filesystem file path, ie

_posts/2012-12-24-delete-a-thing.md

instead of custom defined path property

/stuff/:id

Im a Jekyll/YAML newbie, but seems like path may now be reserved, and therefore overridden. Changing the variable name in the post to something else, ie resturi and the equivalent change in template in index.html fixed the issue.

So in your post, the line:

path: '/stuff/:id'

changed to :

resturi: '/stuff/:id'

and in index.html, changed post.path to post.resturi

<a name='{{ post.url }}' href='#{{ post.url }}'><h2>{% if post.type %}<code><b>{{ post.type }}</b> {{ post.path }}</code> {% endif %}{{ post.title }}</h2></a>
changed to :

<a name='{{ post.url }}' href='#{{ post.url }}'><h2>{% if post.type %}<code><b>{{ post.type }}</b> {{ post.resturi }}</code> {% endif %}{{ post.title }}</h2></a>

@saraiva132
Copy link

This issue still exists, i would either advise you to make a pull request or i can do it myself.

@Dzhuneyt
Copy link
Contributor

@hunvreus @saraiva132 Could you please review #31 if possible? It should solve this ticket.

@vivi90
Copy link

vivi90 commented May 2, 2020

@hunvreus
Copy link
Owner

hunvreus commented May 2, 2020

Looking into it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants