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

Save correct document url from document360 #1413

Merged
merged 1 commit into from
May 2, 2024

Conversation

VikasNeha
Copy link
Contributor

  1. Fixed incorrect document base url
  2. Articles sometime have an external url, save it if available.

Copy link

vercel bot commented May 1, 2024

@VikasNeha is attempting to deploy a commit to the Danswer Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@yuhongsun96 yuhongsun96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good, thanks!

@yuhongsun96 yuhongsun96 merged commit 143b50c into danswer-ai:main May 2, 2024
3 of 4 checks passed
onimsha added a commit to mindvalley/danswer that referenced this pull request May 2, 2024
* Add check to ensure auth is enabled for every endpoint unless explicitly whitelisted

* Swap Index Early (danswer-ai#1353)

* Pre download models (danswer-ai#1354)

* Only Log Index Attempt CC Pair Miscount (danswer-ai#1355)

* Remove Unused Volumes (danswer-ai#1356)

* Web Connector to only allow Global IPs (danswer-ai#1357)

* Personal assistants

* Assistant rework fixes

* Allow separate vespa config server host

* Remove Nested DB Sessions (danswer-ai#1367)

* Vespa Batch Size (danswer-ai#1368)

* fix error: NextRouter was not mounted

* k

* Disabled Connectors List (danswer-ai#1376)

* Add env variable to disable streaming for the DefaultMultiLLM class

* properly yield non-streaming responses

* Rework Disabled Connector options to Whitelist instead (danswer-ai#1386)

* Connectors Whitelist Fix (danswer-ai#1387)

* Fix Tag Integer Enums (danswer-ai#1388)

* Support to loop through all nested subcategories (danswer-ai#1382)

* Fix for parser failing if doc is blank

* Support to loop through all nested child categories

* Fix credential form refresh

* Add token rate limit tables to MIT

* Fix document360 mypy issue

* Fix compose file dependencies

* Set message logging to debug (danswer-ai#1396)

* Fix Citation Sort Tiebreak (danswer-ai#1397)

* Add UI-based LLM selection

* Address 'PGRES_TUPLES_OK and no message from the libpq' issues

* Cancel scheduled indexing attempts on deletion request

* fix(gitignore): properly ignore celery-generated file

* Add image upload capabilities

* Address rate limiting for Notion

* Fix double message send

* Misc UI improvments

* Api key email display for manage/me endpoint (danswer-ai#1409)

* Manage me endpoint key rename (danswer-ai#1410)

* Save correct document url from document360 (danswer-ai#1413)

---------

Signed-off-by: Alex Co <alex.tuan@mindvalley.com>
Co-authored-by: Weves <chrisweaver101@gmail.com>
Co-authored-by: Yuhong Sun <yuhongsun96@gmail.com>
Co-authored-by: Jignesh Solanki <jigneshsolanki92@gmail.com>
Co-authored-by: Mehmet Bektas <mbektasgh@outlook.com>
Co-authored-by: Vikas Neha Ojha <neha.nehavikas.vikas@gmail.com>
Co-authored-by: Davy Peter Braun <543614+dheavy@users.noreply.github.com>
@VikasNeha VikasNeha deleted the document360-article-url branch May 13, 2024 10:46
ptitzlabs pushed a commit to svoi-fr/mirai that referenced this pull request May 16, 2024
ThomasRitaine pushed a commit to ThomasRitaine/chatdoc that referenced this pull request Jun 4, 2024
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.

2 participants