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.

Sign up for GitHub

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

Jump to bottom

[DUOS-2176][risk=no] Remove unused code #1875

Merged

rushtong merged 1 commit into develop from DUOS-1740

Nov 3, 2022

Contributor

rushtong commented Nov 2, 2022 •

edited

Loading

Addresses

https://broadworkbench.atlassian.net/browse/DUOS-2176
Purely a cleanup PR to remove unused API calls and an empty file.

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment


          remove unused code

802bdd5

rushtong requested a review from a team as a code owner

November 2, 2022 12:59

rushtong changed the title ~~[DUOS-1740][risk=no] Remove unused code~~ [DUOS-2176][risk=no] Remove unused code

quazi-h requested review from otchet-broad and JVThomas

November 2, 2022 13:37

otchet-broad approved these changes

View reviewed changes

JVThomas approved these changes

View reviewed changes

rushtong merged commit 6a1cb57 into develop

rushtong deleted the DUOS-1740 branch

November 3, 2022 09:06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet