we need a new display plugin for <opml> though, should be fairly easy to add
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | aggregator-2039277.patch | 3.88 KB | dawehner |
| #8 | 2039277-aggregator_opml-8.patch | 7.21 KB | googletorp |
| #8 | interdiff.txt | 1.08 KB | googletorp |
| #6 | 2039277-aggregator_opml-6.patch | 7.22 KB | googletorp |
| #2 | 2039277-aggregator_opml-2.patch | 4.78 KB | sandhya.m |
Comments
Comment #1
vijaycs85Comment #2
sandhya.m commentedInitial patch...
Comment #3
dawehnerNo views? I haz a sad.
Comment #4
googletorp commented#2: 2039277-aggregator_opml-2.patch queued for re-testing.
Comment #6
googletorp commentedRerolled the patch as is, since #2089635: Convert non-test non-form page callbacks to routes and controllers messed this patch up.
Comment #8
googletorp commentedFixed noticed introduced in the above patchs and make the route name use consistent naming (. notation).
Comment #9
lostkangaroo commentedThis is now blocking #1977608: Update hook_help Aggregator module, which needs to point to this view.
Comment #9.0
lostkangaroo commentedcode..
Comment #10
dawehnerThis patched had a bunch of unrelated changes, probably due to rerolls etc.
Comment #12
dawehner10: aggregator-2039277.patch queued for re-testing.
Comment #13
ParisLiakos commentedyeah, if i find time to write views plugin for OPML i ll open a new issue..lets get rid of the module_load_include ugliness in there for now
Comment #14
xano10: aggregator-2039277.patch queued for re-testing.
Comment #15
webchickCommitted and pushed to 8.x. Thanks!