Closed lanitochka17 closed 8 months ago
Job added to Upwork: https://www.upwork.com/jobs/~01c83a1df166670dd9
Triggered auto assignment to @kevinksullivan (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)Triggered auto assignment to Contributor-plus team member for initial proposal review - @alitoshmatov (External
)
Login - When user sign out, apple&google icon displayed with delay
At BaseLoginForm script loading for AppleSignIn and GoogleSignIn only load when component render, so when network is slow it will take time to load the script
and the button only display when script loaded
Instead of return null until script loaded, we will return a ActivityIndicator
or fake icon button both Apple and Google
POC
https://github.com/Expensify/App/assets/11959869/27c21200-87ff-4ac5-bdd5-c60e3749f909
We can move the loading script for Apple and Goole script to web/index.html
@kevinksullivan, @alitoshmatov Whoops! This issue is 2 days overdue. Let's get this updated quick!
The Google & Facebook Login buttons display with an initial delay from the main login page after logout.
In ' Android: mWeb Chrome', 'GoogleSignIn/index.website.js' is triggered and 'AppleSignIn/index.website.js'. In there, we await a script:
In the above code, we return null - whereas I propose we return a placeholder for the script:
Then repeat the same process for 'GoogleSignIn'.
Add a useState to 'GoogleSignIn':
const [scriptLoaded, setScriptLoaded] = useState(false);
Including a conditional statemement after the script where we render a placeholder div instead of null:
if (scriptLoaded === false) {
return <PlaceholderComponent />; // Render your placeholder component here
}
None.
@kevinksullivan, @alitoshmatov Eep! 4 days overdue now. Issues have feelings too...
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@kevinksullivan, @alitoshmatov 6 days overdue. This is scarier than being forced to listen to Vogon poetry!
Sorry, been busy with holidays. Review it today or tomorrow.
@kevinksullivan, @alitoshmatov Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
@kevinksullivan So the issue here is that we are waiting for a corresponding third party scripts to load, and showing icons only after the scripts are loaded. I would say it is expected behavior. But if we want to change this we have two options:
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@kevinksullivan @alitoshmatov this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!
@kevinksullivan, @alitoshmatov Whoops! This issue is 2 days overdue. Let's get this updated quick!
Waiting for @kevinksullivan 's take on https://github.com/Expensify/App/issues/33414#issuecomment-1873848395
Upon rereading the issue and checking the video, I don't think this is worth addressing. I don't think either solution is clearly superior to what we have today.
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.15-1 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: Applause - Internal Team Slack conversation:
Action Performed:
Expected Result:
When user opens staging.expensify.com or sign out and navigate to staging.expensify.com, apple&google icon must be displayed without delay
Actual Result:
When user sign out, apple&google icon displayed after a second in delay But when user opens staging.expensify.com, apple&google icon displayed without delay
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/78819774/ee62ef91-52cd-4ac1-846c-f8108cd89ddf
View all open jobs on GitHub
Upwork Automation - Do Not Edit