This would make so many things simpler.

CommentFileSizeAuthor
#9 feeds-push-import-2531858-9.patch5.92 KBtwistor
#7 feeds-push-import-2531858-7.patch4.38 KBtwistor
#5 feeds-push-import-2531858-3.patch3.47 KBtwistor
#3 feeds-push-import-2531858-3.patch3.47 KBtwistor
#1 feeds-push-import-2531858-1.patch2.68 KBtwistor

Comments

twistor’s picture

twistor commented
Status: Active » Needs review
StatusFileSize
new feeds-push-import-2531858-1.patch2.68 KB
twistor’s picture

twistor commented

This could also simplify testing by allowing use to generate feed items in memory and pushing them.

twistor’s picture

twistor commented
StatusFileSize
new feeds-push-import-2531858-3.patch3.47 KB

Now with a simple test.

Status: Needs review » Needs work

The last submitted patch, 3: feeds-push-import-2531858-3.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-push-import-2531858-3.patch3.47 KB

Interesting, passes locally.

Status: Needs review » Needs work

The last submitted patch, 5: feeds-push-import-2531858-3.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-push-import-2531858-7.patch4.38 KB

Status: Needs review » Needs work

The last submitted patch, 7: feeds-push-import-2531858-7.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-push-import-2531858-9.patch5.92 KB

Nifty. Fixed a bug in my local install, plus made tests more robust.

  • twistor committed 335d175 on 7.x-2.x 
    Issue #2531858 by twistor: Add a FeedsSource::pushImport() method
twistor’s picture

twistor commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.