- Issue created by @wjackson
- Issue was unassigned.
- Status changed to Needs review
9 months 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 Nepal๐ณ๐ต, Kathmandu
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
7 months ago 6:01pm 19 April 2024 - ๐ณ๐ตNepal sujan shrestha Nepal๐ณ๐ต, Kathmandu
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.