-
Notifications
You must be signed in to change notification settings - Fork 5
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
Checks whether correct params have been passed during OTP verification #304
Conversation
Hello @Manyakalra3! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2022-01-20 05:11:21 UTC |
Codecov Report
@@ Coverage Diff @@
## master #304 +/- ##
==========================================
+ Coverage 97.01% 97.02% +0.01%
==========================================
Files 147 147
Lines 2714 2725 +11
==========================================
+ Hits 2633 2644 +11
Misses 81 81
Continue to review full report at Codecov.
|
Fixes #301
Summary
Tests are written for testing whether Params (mobile and otp) have been passed during the OTP verification.
Test Plan
If adding any new environment variable:- update
- update the environment variables for staging and production
docs/ENV.md