Ticket #2177 (closed enhancement: fixed)

Opened 6 years ago

Last modified 5 years ago

Dashboard's Feed URL should be pluggable

Reported by: davidhouse Owned by: rob1n
Priority: normal Milestone: 2.3
Component: Administration Version: 2.0
Severity: minor Keywords: has-patch commit
Cc:

Description

We should allow plugins to change the feed that the dashboard uses. Patch attached, filter is 'dash_feed_url'.

Attachments

wp-dashboard-feeds.diff Download (1.4 KB) - added by technosailor 5 years ago.
2177.diff Download (2.5 KB) - added by rob1n 5 years ago.

Change History

comment:1   davidhouse — 6 years ago

  • Keywords bg|commit added

comment:2   Nazgul — 5 years ago

  • Keywords has-patch commit added; bg|has-patch bg|commit removed
  • Milestone set to 2.1

comment:3   matt — 5 years ago

  • Milestone changed from 2.1 to 2.2

comment:4   markjaquith — 5 years ago

Patch needs refresh.

comment:5   rob1n — 5 years ago

  • Owner changed from anonymous to rob1n
  • Status changed from new to assigned

Might as well go the whole 9 yards. Adds filters to all the URI's.

comment:6   rob1n — 5 years ago

  • Type changed from defect to enhancement

comment:7   rob1n — 5 years ago

  • Keywords dashboard feed filter has-patch commit removed

rboren suggested more generic feed filter names. working on new patch.

comment:8   rob1n — 5 years ago

  • Keywords has-patch commit added

comment:9   ryan — 5 years ago

  • Owner changed from rob1n to ryan
  • Status changed from assigned to new

comment:10   rob1n — 5 years ago

  • Keywords commit removed
  • Milestone changed from 2.2 to 2.3

comment:11   rob1n — 5 years ago

  • Owner changed from ryan to rob1n

comment:12   markjaquith — 5 years ago

I'd rather if the feed hooks weren't named according to the feed they currently show. That could change in the future. how about _primary and _secondary or something generic like that?

technosailor — 5 years ago

comment:13   technosailor — 5 years ago

Since we just implemented this, it makse no sense not to add a refreshed patch. Adds 5 hooks.

  • primary_dash_rssurl
  • primary_dash_title
  • secondary_dash_rssurl
  • secondary_dash_title
  • secondary_dash_url

comment:14   rob1n — 5 years ago

  • Status changed from new to assigned

How about incoming links?

comment:15   rob1n — 5 years ago

  • Status changed from assigned to new

rob1n — 5 years ago

comment:16   rob1n — 5 years ago

  • Keywords commit added
  • Status changed from new to assigned

comment:17   rob1n — 5 years ago

  • Status changed from assigned to closed
  • Resolution set to fixed

(In [5346]) Filter dashboard URL's. fixes #2177

comment:18   technosailor — 5 years ago

Dammit, Rob1n... why do you have to work on Sunday? Now you'll get the "Props rob1n" instead of me! :-p

Good changeset anyway, ha! :)

comment:19   technosailor — 5 years ago

Well, it's good except for you need this guys patch [4208]

Note: See TracTickets for help on using tickets.