Add ability to Test feeds and perform manual pulls via AJAX with log response #15
Conversation
…ck array of XML items This abstracts the field mapping out of do_pull() which is super useful, but also mainly for testing purposes to be as accurate as possible
…hird party plugins One more layer of abstraction, to make the world a better place for third party plugins to extend Feed Pull for their own mapping
Add ability to Test feeds and AJAX load the log when doing manual pull
Add support for user agent override in the request args for wp_remote_get
Check if $feed is strictly false to detect if feed was not parsed
Conflicts: build/css/post-admin.css build/css/post-admin.min.css build/js/post-admin.min.js includes/class-fp-pull.php
|
Opening this PR up so it'll hook into the tests. I'm still working on improving the test output so it's more readable and friendly. Thinking about opening up a modal with the testing response. |
|
Added the patch that fixes it so when you run manual pulls, it won't check if the automatic feed pull cron is enabled or not -- it just bypasses it and lets people run manual pulls as they wish. |
|
@sc0ttkclark I think this is a really cool idea, but I'm not sure how I feel about merging the var_dump to sidebar thing into production. |
|
Consider this a work in progress until I nail down the UI for seeing the test return data, I mentioned above I'm planning on displaying it a bit nicer :) |
|
Sounds good! |
|
@sc0ttkclark Cannot thank you enough for making these changes. The ability to preview field mappings saved me so much time. |
Loads log via AJAX on test and lets you do manual pulls and get instant logs back from AJAX response.
Includes refactor in AJAX and Pull classes that abstracts more things out for reusability and make it possible to test more cases.