Skip to content

Commit

Permalink
Remove fetch polyfill from FidesJS (#5026)
Browse files Browse the repository at this point in the history
  • Loading branch information
gilluminate authored Jun 27, 2024
1 parent 1f464b5 commit 868c4d6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 10 deletions.
6 changes: 3 additions & 3 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,12 @@ The types of changes are:

### Changed
- Navigation changes. 'Management' was renamed 'Settings'. Properties was moved to Settings section. [#5005](https://github.com/ethyca/fides/pull/5005)


### Changed
- Changed discovery monitor form behavior around execution date/time selection [#5017](https://github.com/ethyca/fides/pull/5017)
- Changed integration form behavior when errors occur [#5023](https://github.com/ethyca/fides/pull/5023)

### Removed
- Removed the `fetch` polyfill from FidesJS (#5026)[https://github.com/ethyca/fides/pull/5026]

### Fixed
- Fixed intermittent connection issues with Redshift by increasing timeout and preferring SSL in test connections [#4981](https://github.com/ethyca/fides/pull/4981)

Expand Down
7 changes: 0 additions & 7 deletions clients/privacy-center/pages/api/fides-js.ts
Original file line number Diff line number Diff line change
Expand Up @@ -271,13 +271,6 @@ export default async function handler(

const script = `
(function () {
// This polyfill service adds a fetch polyfill only when needed, depending on browser making the request
if (!window.fetch) {
var script = document.createElement('script');
script.src = 'https://polyfill.io/v3/polyfill.min.js?features=fetch';
document.head.appendChild(script);
}
// Include generic fides.js script and GPP extension (if enabled)
${fidesJS}${fidesGPP}${
customFidesCss
Expand Down

0 comments on commit 868c4d6

Please sign in to comment.