DOP-3996: Chatbot server url #880
Conversation
| @@ -5,6 +5,11 @@ import { css } from '@emotion/react'; | |||
| import { theme } from '../theme/docsTheme'; | |||
| import 'react-loading-skeleton/dist/skeleton.css'; | |||
|
|
|||
| const CHATBOT_SERVER_BASE_URL = | |||
| process.env.SNOOTY_ENV === 'production' || process.env.SNOOTY_ENV === 'dotcomprd' | |||
There was a problem hiding this comment.
Would love a second pair of eyes and confirmations on the fact that SNOOTY_ENV's values could all be 'production' and 'dotcomprd'
Double-checked our current task definitions and it seems like these are the only 2 prod-based values we'd expect
| @@ -5,6 +5,11 @@ import { css } from '@emotion/react'; | |||
| import { theme } from '../theme/docsTheme'; | |||
| import 'react-loading-skeleton/dist/skeleton.css'; | |||
|
|
|||
| const CHATBOT_SERVER_BASE_URL = | |||
| process.env.SNOOTY_ENV === 'production' || process.env.SNOOTY_ENV === 'dotcomprd' | |||
There was a problem hiding this comment.
I know this PR is already approved and merged, but I just remembered that I should have asked this:
Does the chatbot team want us to use the staging API for GitHub staging jobs and the production API for only production deployments? If so, we might need to update this logic to exclude process.env.SNOOTY_ENV === 'production'. @caesarbell might have some insight.
There was a problem hiding this comment.
oo @rayangler just getting fully caught up on my email and saw this! I think I might be confused at what SNOOTY_ENV === 'production' would mean in this case? Is writers staging using the value "production"?
There was a problem hiding this comment.
Is writers staging using the value "production"?
Yes, I believe so, based on its value in this task definition. SNOOTY_ENV can be seen as representative of the overall environment that Snooty is being ran within the context of the Autobuilder, with "production" being "prd" and "dotcomprd" being "dotcomprd".
There was a problem hiding this comment.
Ok, great catch. Thank you! I'll open up a PR!
Stories/Links:
DOP-3996
Notes:
The chatbot team would like our non-prod environments to point the Chatbot towards their staging link. This uses the SNOOTY_ENV to determine this.
Would love a second pair of eyes and confirmations on the fact that SNOOTY_ENV's values could all be 'production' and 'dotcomprd'