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

10531: Load legacy pdfjs if legacy browser #5627

Merged

Conversation

Mwindo
Copy link
Contributor

@Mwindo Mwindo commented Dec 4, 2024

See #5567 for comments. We have:

  1. Regression tested on modern browsers
  2. Tested on one older machine in the "Goldilocks" zone (older than ES2022, but not so old that the app doesn't run).

The hope is to see a noticeable reduction in the number of pdf upload errors prod support is receiving, since most (maybe all? 🤞) are related to older browsers.

@Mwindo Mwindo marked this pull request as ready for review December 4, 2024 19:49
@jimlerza jimlerza merged commit 6a6c54e into ustaxcourt:staging Dec 4, 2024
44 checks passed
@jimlerza jimlerza deleted the load-legacy-pdfjs-if-legacy-browser branch December 4, 2024 22:04
@jimlerza jimlerza mentioned this pull request Dec 6, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants