-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[No QA] Remove unused methods and exports #3016
Conversation
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.
Lint is failing b/c of an unused import
@@ -119,7 +119,6 @@ class AuthScreens extends React.Component { | |||
User.getBetas(); | |||
fetchAllReports(true, true); | |||
fetchCountryCodeByRequestIP(); | |||
BankAccounts.fetchBankAccountList(); |
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.
Looks like the BankAccounts
import at the top of the file can also be removed
Ooops, updated to remove the import.
…On Wed, May 19, 2021 at 3:07 PM Rocio Perez ***@***.***> wrote:
***@***.**** approved this pull request.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#3016 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJMAB5ZJEH6ZQA6TIJN3F3TOQSCJANCNFSM45FKVYXQ>
.
|
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
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
This is just some cleanup that I came across
Tests / QA Steps
No QA - Normal regression steps will catch anything
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android