[PR #48] [MERGED] Note that WEB_DOMAIN needs to include the protocol #55

Closed
opened 2026-02-15 17:05:54 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/onyx-dot-app/documentation/pull/48
Author: @swiknaba
Created: 12/22/2023
Status: Merged
Merged: 12/22/2023
Merged by: @Weves

Base: mainHead: patch-1


📝 Commits (1)

  • 460e4f0 Note that WEB_DOMAIN needs to include the protocol

📊 Changes

1 file changed (+1 additions, -0 deletions)

View changed files

📝 google_oauth_setup.mdx (+1 -0)

📄 Description

Omitting the protocol results in an OAuth error, which tripped up several people, including my self.

See thread in Slack.

Thanks for building Danswer!

Feel free to do any edits to my PR if you think this should be in a different place :)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/onyx-dot-app/documentation/pull/48 **Author:** [@swiknaba](https://github.com/swiknaba) **Created:** 12/22/2023 **Status:** ✅ Merged **Merged:** 12/22/2023 **Merged by:** [@Weves](https://github.com/Weves) **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`460e4f0`](https://github.com/onyx-dot-app/documentation/commit/460e4f0aeb92fa1755b3be997f02867379274335) Note that WEB_DOMAIN needs to include the protocol ### 📊 Changes **1 file changed** (+1 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `google_oauth_setup.mdx` (+1 -0) </details> ### 📄 Description Omitting the protocol results in an OAuth error, which tripped up several people, including my self. See thread in [Slack](https://danswer.slack.com/archives/C056265VB1N/p1703180841734139). Thanks for building Danswer! Feel free to do any edits to my PR if you think this should be in a different place :) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 17:05:54 -05:00
yindo closed this issue 2026-02-15 17:05:55 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: onyx-dot-app/documentation#55