Skip to:
Content

Opened 6 years ago

Last modified 3 years ago

#2305 new enhancement

Automatic Feed Links

Reported by: alex-ye Owned by: netweb
Milestone: 2.7 Priority: normal
Severity: normal Version: 2.3
Component: API - Feeds Keywords: needs-docs needs-codex needs-refresh
Cc: nashwan.doaqan@…, trisha@…

Description

Another Idea for bbPress is to have an Automatic Feed Links such the WordPress feeds system.. So the forum/topic/user feed links displayed automaticly in the head tag.

In also the developer could remove those links easily, So introduce this feature as a theme feature will be a good idea

add_theme_support('bbp-automatic-feed-links');

Attachments (1)

2305.diff (3.8 KB) - added by netweb 5 years ago.

Download all attachments as: .zip

Change History (13)

#1 @johnjamesjacoby
6 years ago

  • Milestone changed from Awaiting Review to 2.4

Selfishly, I've never found links to feeds to be super helpful, so I never included them in the UI. I agree we probably should, though I'm not sure this is the approach I'd take.

I think I'd rather we had a better forum-meta and topic-meta solution for outputting "stuff" like this, in the default template files, as some kind of template part. I'm not entirely sure what that would look like yet, but that's what I'm envisioning for now.

#2 @alex-ye
6 years ago

  • Cc nashwan.doaqan@… added

Maybe the feeds is not "super helpful" but it still "helpful" form many users :)
Interesting to see your suggestion about the approach.

#3 @johnjamesjacoby
6 years ago

  • Milestone changed from 2.4 to 2.5

No time. Moving to 2.5.

#4 @netweb
5 years ago

  • Milestone changed from 2.5 to 2.6

No time. Moving to 2.6.

#5 @trishasalas
5 years ago

  • Cc trisha@… added

This ticket was mentioned in IRC in #bbpress-dev by netweb. View the logs.


5 years ago

@netweb
5 years ago

#7 @netweb
5 years ago

  • Keywords has-patch added; needs-patch removed

So I just found 2305.diff, I created a couple of months back, and with a quick refresh against /trunk
(It needs some PHPDOC and testing with/without forum prefix etc.)

I think I stopped working on this as it was only adding feed links to <head> for RSS discovery, per the original ticket description, then JJJ mentions UI and template parts and that is when I stopped looking at this.

I can't picture (as JJJ mentions above) what said UI and template parts would look like, if we put that part of the equation in another ticket we can have RSS feed links in the HTML HEAD with just 2305.diff.

Add this to your functions.php to enable bbPress RSS Feeds 'auto discovery' links to the HTML <head>

// Adds RSS feed links to <head> for bbPress topics, replies and topic tags.
add_theme_support( 'bbp-automatic-feed-links' );

#8 @netweb
5 years ago

  • Keywords needs-docs needs-codex added

This ticket was mentioned in IRC in #bbpress-dev by netweb. View the logs.


5 years ago

#10 @netweb
5 years ago

  • Owner set to netweb

#11 @johnjamesjacoby
5 years ago

  • Milestone changed from 2.6 to 2.7

Looks good. Likely needs more thought around whether including new feed links in everyone's sites by default is something we want to commit to.

Moving to 2.7.

#12 @netweb
3 years ago

  • Keywords needs-refresh added; has-patch removed

Related:

  • #2292 RSS feeds missing Some HTTP Headers
  • #2901 Use modern feed Content-Type

A quick look at the above tickets and the current 2305.diff patch iterating with feed_content_type( 'rss2' ) and WordPress' feed helper functions would make things a little saner.

Note: See TracTickets for help on using tickets.