-
Notifications
You must be signed in to change notification settings - Fork 6
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
Systest 9779 #137
base: dev
Are you sure you want to change the base?
Systest 9779 #137
Conversation
Issue with linting or unit tests detected.
For more information on our linting policies, please see our Linting-Guide. |
src/pubSubClient.js
Outdated
} else { | ||
logger.error('Failed to initialize a WS connection...', 'initialize'); | ||
} | ||
logger.error('Failed to initialize a WS connection...', event); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's leave this as "info". It will only technically be an error if someone wants to connect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed
src/pubSubClient.js
Outdated
reject(false); | ||
}); | ||
}).catch((error) => { | ||
logger.error('Continuing without PubSub due to WS connection failure.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above RE: "info"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed
|
Description
Enter a detailed description of the PR here, including the nature and purpose of the changes.
JIRA Ticket
Include a link to the JIRA Ticket linked to this PR.
Issue-0000
Additional Info (Test steps / Steps to reproduce etc.)
Details on how to test the feature