-
Notifications
You must be signed in to change notification settings - Fork 3
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
[#IOPID-1449] enable strict=true
on typescript config
#307
Conversation
Jira Pull Request LinkThis Pull Request refers to the following Jira issue IOPID-1449 |
/azp run |
No pipelines are associated with this pull request. |
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.
Small comments - Part 1 :)
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.
Small comments - Part 2/2.
// NOTE: if the following check is changed make sure to pass add the name field | ||
// to CreateProfile method which also calls this orchestrator | ||
if (isProfileEmailChanged) { | ||
if (isProfileEmailChanged && newProfile.email) { |
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.
Although it's unlikely, if the email changes to undefined, could we raise an 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.
Do you mean just for monitoring?
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.
Yes, just to ensure it never happen.
…o avoid Mixed type incompatibility.
Quality Gate passedIssues Measures |
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.
🚀
List of Changes
Code refactory after enabling the
strict=true
option intsconfig
.Motivation and Context
Having a
strict=false
option for Typescript transpiling can generate application and logic errors to handle properlyundefined
ornull
properties. This setting helps to prevent bugs in production.How Has This Been Tested?
build
and unit testScreenshots (if appropriate):
Types of changes