feat(jira): support vault id #41
Conversation
Djiit
commented
Oct 29, 2024
Djiit
commented
- feat(jira): support vaultid
- fix(jira): add missing dependency in package.json
| @@ -235,7 +248,7 @@ function formatDescription( | |||
| ): string { | |||
| return ( | |||
| description?.content | |||
| .map((c) => c.content.map((cc) => cc.text).join("")) | |||
| .map((c) => c.content?.map((cc) => cc.text).join("")) | |||
There was a problem hiding this comment.
Looks like some of our tickets had objects like type: rule in it, without any content.
![dust-agent-dev[bot]](https://avatars.githubusercontent.com/in/1108125?s=60&v=4){kind=small}
|
|
||
| const missingEnvVars = requiredEnvVars.filter(varName => !process.env[varName]); | ||
| const missingEnvVars = requiredEnvVars.filter( |
There was a problem hiding this comment.
The line formatting seems off. The lambda function should be on the same line as the filter method for better readability.
| const DUST_API_KEY = process.env.DUST_API_KEY; | ||
| const DUST_WORKSPACE_ID = process.env.DUST_WORKSPACE_ID; | ||
| const DUST_VAULT_ID = process.env.DUST_VAULT_ID; | ||
| const DUST_DATASOURCE_ID = process.env.DUST_DATASOURCE_ID; | ||
|
|
||
| const requiredEnvVars = [ | ||
| 'JIRA_SUBDOMAIN', | ||
| 'JIRA_EMAIL', | ||
| 'JIRA_API_TOKEN', | ||
| 'DUST_API_KEY', | ||
| 'DUST_WORKSPACE_ID', | ||
| 'DUST_DATASOURCE_ID' | ||
| "JIRA_SUBDOMAIN", | ||
| "JIRA_EMAIL", | ||
| "JIRA_API_TOKEN", | ||
| "DUST_API_KEY", | ||
| "DUST_WORKSPACE_ID", | ||
| "DUST_VAULT_ID", | ||
| "DUST_DATASOURCE_ID", | ||
| ]; | ||
|
|
||
| const missingEnvVars = requiredEnvVars.filter(varName => !process.env[varName]); | ||
| const missingEnvVars = requiredEnvVars.filter( |
There was a problem hiding this comment.
Please ensure consistent use of single quotes for strings to adhere to coding standards. Change instances of double quotes to single quotes.
| ]; | ||
|
|
||
| const missingEnvVars = requiredEnvVars.filter(varName => !process.env[varName]); | ||
| const missingEnvVars = requiredEnvVars.filter( |
There was a problem hiding this comment.
The line formatting seems off. The lambda function should be on the same line as the filter method for better readability.
| ]; | ||
|
|
||
| const missingEnvVars = requiredEnvVars.filter(varName => !process.env[varName]); | ||
| const missingEnvVars = requiredEnvVars.filter( |
There was a problem hiding this comment.
Please ensure consistent use of single quotes for strings to adhere to coding standards. Change instances of double quotes to single quotes.
