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

fix: remove redirection by changing /play route to direct routing #3269

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

SaeWooKKang
Copy link
Contributor

While looking at the ways to improve the rendering performance of the /play page, I discovered a redirect occurring when navigating to the page. Please let me know if you have any problems. Thank you always.

What this PR solves

  • remove redirection by changing /play route to /play/

Expected Benefits

  • Eliminates ~570ms delay on /play page load during slow network conditions
스크린샷 2024-11-05 오후 2 53 13

AS-IS

2024-11-05.2.13.53.mov

TO-BE

2024-11-05.2.11.24.mov

@jakebailey jakebailey added the deploy-preview Enables automatic deployments to preview environments on a PR label Nov 8, 2024
Copy link
Contributor

github-actions bot commented Nov 8, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-plant-05c166c10-3269.centralus.5.azurestaticapps.net

@jakebailey jakebailey merged commit 078f7ee into microsoft:v2 Nov 8, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-preview Enables automatic deployments to preview environments on a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants