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.

Sign up for GitHub

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

Jump to bottom

Listing links in RSS point to link://listing/... #952

Closed
roman-yepishev opened this issue Dec 4, 2013 · 2 comments
Closed

Listing links in RSS point to link://listing/... #952

roman-yepishev opened this issue Dec 4, 2013 · 2 comments
Assignees
@ralsina

Comments

@roman-yepishev
Copy link

STR:

  1. Add a new listing to the page (e.g. http://rtg.in.ua/blog/yandex-disk-apparmor/index.rst)
  2. Look at the generated RSS:
<a class="reference external"
          href="link://listing/usr.bin.yandex-disk"
>usr.bin.yandex-disk</a>
@ralsina
Copy link
Member

ralsina commented Jan 27, 2014

Right, we are not doing the link:: fix on RSS feeds. Looking into it.

@ghost ghost assigned ralsina Jan 27, 2014
@ralsina
Copy link
Member

ralsina commented Jan 27, 2014

There are 2 ways to fix this:

  1. I can make the listing directive not use link://

PRO: easy
CON: Doesn't fix any other use of link://

  1. I can move generic_rss from utils into the Nikola class and make it post-process the HTML like we do for actual posts

PRO: Fixes the root cause
CON: largish refactor, changes API?

I am leaning towards 2)

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

@ralsina ralsina

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ralsina @roman-yepishev