Skip to content

Remove es5.js and browser check from html files#12494

Open
jna4k wants to merge 1 commit into
geosolutions-it:masterfrom
jna4k:remove-es5js-browser-check
Open

Remove es5.js and browser check from html files#12494
jna4k wants to merge 1 commit into
geosolutions-it:masterfrom
jna4k:remove-es5js-browser-check

Conversation

@jna4k

@jna4k jna4k commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR removes the es5.js file that was being loaded from a external CDN. It also removes a browser check that redirected the user to the "unsupported browser page" if the user were using a browser other than chrome, firefox, edge or safari. The PR also removes said unspported browser page.

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?
We are currently using a 3rd party javascript file from a CDN, and uses it to check the users browser version, if the user is using Internet Explore they will be redirected to a unsupported browser page.

#12477

What is the new behavior?
We will no longer load the es5.js file and the we will no longer do the browser check. This means if the user is using a unspported browser, the site will simply fail.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

@cla-bot cla-bot Bot added the CLA Ready label Jun 15, 2026
@jna4k jna4k closed this Jun 16, 2026
@jna4k jna4k force-pushed the remove-es5js-browser-check branch from 4defdf7 to bd137d9 Compare June 16, 2026 08:14
@jna4k jna4k reopened this Jun 16, 2026
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.

1 participant