Skip to content

799 help api#41

Merged
rakeshSgr merged 16 commits intodevfrom
799-helpApi
Sep 7, 2022
Merged

799 help api#41
rakeshSgr merged 16 commits intodevfrom
799-helpApi

Conversation

@nevil-mathew
Copy link
Collaborator

No description provided.

@nevil-mathew nevil-mathew requested a review from rakeshSgr July 18, 2022 06:32
src/.env.sample Outdated
API_DOC_URL = '/api-doc'

#Enable email for reported issue.
ENABLE_EMAIL_FOR_REPORTED_ISSUE = true
Copy link
Collaborator

Choose a reason for hiding this comment

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

@nevil-mathew change to this ENABLE_EMAIL_FOR_REPORT_ISSUE

src/.env.sample Outdated
ENABLE_EMAIL_FOR_REPORTED_ISSUE = true

#Email id of the support team.
SUPPORT_EMAIL_ID = 'support@xyz.com'
Copy link
Collaborator

Choose a reason for hiding this comment

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

@nevil-mathew can we take comma-separated email ids?

src/.env.sample Outdated
SUPPORT_EMAIL_ID = 'support@xyz.com'

#Email template code for reported issue.
REPORTED_ISSUE_EMAIL_TEMPLATE_CODE = 'user_issue_reported'
Copy link
Collaborator

Choose a reason for hiding this comment

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

@nevil-mathew can we change to this REPORT_ISSUE_EMAIL_TEMPLATE_CODE ?

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

API_DOC_URL = '/api-doc'

#Enable email for reported issue.
ENABLE_EMAIL_FOR_REPORT_ISSUE = true
Copy link
Collaborator

Choose a reason for hiding this comment

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

@nevil-mathew Can add these in envVariable.js

@rakeshSgr rakeshSgr merged commit d543fd1 into dev Sep 7, 2022
tech-infra pushed a commit that referenced this pull request Feb 14, 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