-
Notifications
You must be signed in to change notification settings - Fork 55
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
Snyk May Review #7702
Snyk May Review #7702
Conversation
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.
tested on dev4 and lgtm!
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.
Smoke tested on dev4 -- LGTM! Thank you for all your work on this @shanice-skylight
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.
LGTM! Thanks Shanice!
* fixed 89 & 284 * fixed 287 & 770 * testing modified package back to original * fixed backages error * updated all high and critical * modified runtime class * modified liquibase version to 4.20.0 to fix error * updated liquibase version * updated liquibase to 4.27.0 * upgrade libraries * added to build.gradle * modified source Compatibility to fix Deprecated Gradle features error * modified to use the java { } * change source version back to original * changed org.postgresql:postgresql back to version 42.6.1 to fix vulnerability
BACKEND PULL REQUEST
Changes Proposed
Testing
A smoke test of basic functionality should be accomplished before merging.
This PR has already been verified in
dev4
.