- Issue created by @wjackson
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 11:49pm 16 February 2024 - 🇺🇸United States wjackson Aiken, SC
The attached patch is provided for ease of access and is derived from the merge request in comment #2 ✨ Feeds Integration Needs review . This patch/MR should allow for Feeds integration. I hope this helps, but let me know if there are any questions.
Thank you!
- First commit to issue fork.
- 🇺🇸United States earthday47 New York
During testing I found that the import fails if the value is empty/blank. So I added a check for non-null to the plugin.
- 🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal
sujan shrestha → made their first commit to this issue’s fork.
-
sujan shrestha →
committed 881c6278 on master authored by
wjackson →
Issue #3422053 by wjackson, earthday47: Feeds Integration
-
sujan shrestha →
committed 881c6278 on master authored by
wjackson →
- Status changed to Fixed
10 months ago 6:01pm 19 April 2024 - 🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal
Hi @wjackson @earthday47 Thank you for the patch! Your contribution is greatly appreciated. With this fix, we can now integrate the Feeds import with content that includes fields from the Simple Time Field module.
- 🇺🇸United States earthday47 New York
You got it! I just pushed up another small fix after more testing. Hopefully that will do it.
Automatically closed - issue fixed for 2 weeks with no activity.