Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove redundant .docx loader (closes #1716) + update how_to_guides.rst #1849

Closed
wants to merge 8 commits into from
Closed

Remove redundant .docx loader (closes #1716) + update how_to_guides.rst #1849

wants to merge 8 commits into from

Conversation

klein-t
Copy link
Contributor

@klein-t klein-t commented Mar 21, 2023

In #1716 , it was identified that there were two .py files performing similar tasks. As a resolution, one of the files has been removed, as its purpose had already been fulfilled by the other file. Additionally, the init has been updated accordingly.

Furthermore, the how_to_guides.rst file has been updated to include links to documentation that was previously missing. This was deemed necessary as the existing list on https://langchain.readthedocs.io/en/latest/modules/document_loaders/how_to_guides.html was incomplete, causing confusion for users who rely on the full list of documentation on the left sidebar of the website.

@klein-t
Copy link
Contributor Author

klein-t commented Mar 22, 2023

I messed up with git so I'll open a new PR

@klein-t klein-t closed this Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants