[community] [Bugfix] base_o365 document loader metadata needs to be JSON serializable

[MacanPN]

In order for indexer to work, all metadata in the documents need to be JSON serializable. Timestamps are not.

See here: https://github.com/MacanPN/langchain/blob/6a44b97bca6d4adafbbdedf42b51aef14f5780cd/libs/core/langchain_core/indexing/api.py#L91

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
langchain	⬜️ Ignored (Inspect)	Visit Preview		Oct 2, 2024 0:27am

[MacanPN]

@baskaryan , @efriis, @eyurtsev, @ccurme, @vbarda, @hwchase17 is anyone interested in picking up this tiny bugifx? Indxing documents from O365 loader does not work without it. Thanks!

[MacanPN]

Pretty pretty please @baskaryan , @efriis, @eyurtsev, @ccurme, @vbarda, @hwchase17 :-)

[efriis]

Hey! Here's a link to the review policy: https://python.langchain.com/docs/contributing/review_process/

Status is currently needs support

[petergoldstein]

Would love to see this fixed. It's small and a relatively obvious bug.

[avalon-swain]

This would be a great fix

[MacanPN]

@efriis Could you please let me know what else can I do? There's 6 upvotes on the PR now, it's literally 2 lines of code (10 characters) and it keeps O365 loader from being index-able by Langchain Indexing.