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

Goodbye, IE 11 #55142

Merged
merged 2 commits into from
Aug 9, 2021
Merged

Goodbye, IE 11 #55142

merged 2 commits into from
Aug 9, 2021

Conversation

noahtallen
Copy link
Contributor

@noahtallen noahtallen commented Aug 3, 2021

Changes proposed in this Pull Request

Enable redirect for fallback browsers in all environments. Follow-up from #53221, which implemented the redirect for development and some testing environments. I also took the opportunity to remove documentation referencing IE11 and the fallback build.

See #53222 for follow-up tasks.

Testing instructions

Use calypso.live for the following:

  • Verify that IE 11 is redirected to /browsehappy. Browserstack works well for a quick test. (Just load the calypso.live link in IE 11. You can also test the "continue anyways" path and ensure it does not redirect after that.
  • Verify that supported browsers are not redirected. (Load calypso.live in something like chrome or firefox and ensure that you do not get redirected.

We will have to double check this behavior after shipping once the deployment makes it to staging, and then again after it gets to prod, as this code change modifies environment configuration.

@noahtallen noahtallen requested review from a team August 3, 2021 21:59
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Aug 3, 2021
@noahtallen noahtallen self-assigned this Aug 3, 2021
@github-actions
Copy link

github-actions bot commented Aug 3, 2021

@matticbot
Copy link
Contributor

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

@noahtallen noahtallen force-pushed the enable-fallback-redirect-prod branch 2 times, most recently from dcb9451 to a3da43c Compare August 5, 2021 19:10
@noahtallen noahtallen requested review from scinos and a team August 5, 2021 19:44
@worldomonation
Copy link
Contributor

Reminder to myself to scan and remove any parts about IE11 in the test/e2e side of things.

@noahtallen noahtallen force-pushed the enable-fallback-redirect-prod branch from a3da43c to 04ffded Compare August 9, 2021 19:58
Copy link
Contributor

@griffbrad griffbrad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Tests well for me in IE11 and a number of modern browsers!

@noahtallen noahtallen merged commit d40b055 into trunk Aug 9, 2021
@noahtallen noahtallen deleted the enable-fallback-redirect-prod branch August 9, 2021 20:13
@github-actions github-actions bot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Aug 9, 2021
flootr added a commit that referenced this pull request Aug 10, 2021
flootr added a commit that referenced this pull request Aug 10, 2021
@noahtallen noahtallen restored the enable-fallback-redirect-prod branch August 26, 2021 21:02
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.

5 participants