-
Notifications
You must be signed in to change notification settings - Fork 0
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
Reuse Tokens & Misc. Changes #97
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## development #97 +/- ##
===============================================
- Coverage 81.59% 81.14% -0.46%
===============================================
Files 15 15
Lines 864 907 +43
===============================================
+ Hits 705 736 +31
- Misses 159 171 +12 ☔ View full report in Codecov by Sentry. |
- Autofocus and Enforce Phone Validation in non-test mode - Introduce a variable to determine if running flutter test - Dialog return values as ints, instead of type conversion - Implement full screen dialog on mobile view - Change phone number input country code to bottom sheet to prevent loading all flags on widget mount. Also had issues on mobile.
Increases the contrast for disabled inputs from 1.2:1 to 4.25:1, recommended contrast is 3:1 to meet accessibility standards
Visit the preview URL for this PR (updated for commit 4ee5cbe): https://angeleno-my-account-dev-402917--pr97-ghi63-reuse-token-fm8dh8wy.web.app (expires Thu, 11 Apr 2024 22:28:10 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: ef7f92b444e46d3ebe3d59054b9a245afee849f5 |
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 approved
What does this PR do?
The main changes in this PR is the reusing of a token as long as it has not expired.
Minor changes:
Background info:
We were creating new tokens on every request
Which issue(s) is/are related to this PR?
This PR is/are related to issue(s)
Closes #63