search

tlovett1 / feed-pull

Pull feeds into WordPress
29 stars 13 forks source link

Add ability to Test feeds and AJAX load the log when doing manual pull #4

Closed sc0ttkclark closed 9 years ago

sc0ttkclark commented 9 years ago

screen shot 2015-01-30 at 10 25 10 pm

And on clicking of Manual pull, it now AJAX refreshes the log meta box with the latest from the manual pull.

I also fixed an issue with the meta box #fp_manual_pull that the JS was targeting unintentionally for the button with the same ID. Changed button ID to #fp_manual_pull_run

sc0ttkclark commented 9 years ago

I should also note that to be able to do all this, I had to abstract the feed fetching functionality into it's own get_feed_posts method and give it public access. do_pull still has private access and utilizes this new abstracted method just fine.

The constructor now accepts a false, and if false it will not run do_pull (null still runs all feeds).

sc0ttkclark commented 9 years ago

This does not send the current feed settings from the form, it still requires saving, just like Do Feed Pull. I chose not to change that functionality there.

sc0ttkclark commented 9 years ago

Going to close and rebased (w/ changes as requested) on develop