Skip to content
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

⚑ Improves login performance by ~4 seconds! #12

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nrmnqdds
Copy link

@nrmnqdds nrmnqdds commented Oct 4, 2023

🚧 What this PR do?

πŸ“ Docs:

  • Rename authenticate to login for more descriptive yet user friendly for user to type

Screenshot 2023-10-04 113010

⚑ Authentication speed:

  • Uses minimal args when launching browser by disabling unnecessary task
  • Disables stylesheets, fonts and images files for faster page loading

πŸ§ͺ Ran the test 3 times and the result:

Screenshot 2023-10-04 111609

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant