-
Notifications
You must be signed in to change notification settings - Fork 39
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
[MDS-5715] Project Agent #2918
[MDS-5715] Project Agent #2918
Conversation
…ther files, unused imports.
…r address, allow some international addresses, put on validation and make a new post code method that works with hooks structure. Fixed an alignment issue I noticed on Actions menu on core. Don't navigate to a new page on the form if there are errors
export const getFormattedProjectSummary = createSelector( | ||
[getProjectSummary, getProject], | ||
(summary, project) => { | ||
let formattedSummary = { ...summary, authorizationOptions: [] }; | ||
const agent = formatProjectSummaryAgent(summary.agent); |
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.
Will this throw an error if agent is null or undefined? I can't remember if strict null checks is enabled in our code base or not.
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.
agent doesn't have to be defined, but summary does. I think it must always be defined because it throws no errors on new entries.
c7f168a
to
8570427
Compare
Quality Gate failed for 'bcgov-sonarcloud_mds_minespace-web'Failed conditions 0.0% Coverage on New Code (required ≥ 80%) |
Quality Gate failed for 'bcgov-sonarcloud_mds_common'Failed conditions 0.0% Coverage on New Code (required ≥ 80%) |
Quality Gate passed for 'bcgov-sonarcloud_mds_core-web'Kudos, no new issues were introduced! 0 New issues |
Quality Gate failed for 'bcgov-sonarcloud_mds_core-api'Failed conditions 0.0% Coverage on New Code (required ≥ 80%) |
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.
🙌🏻
Objective
MDS-5715
MDS-5739
Why are you making this change? Provide a short explanation and/or screenshots