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 and Kelsey-Ethyca committed Jun 28, 2024
1 parent 9cac14b commit ceb1c60
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ The types of changes are:

## [2.39.1](https://github.com/ethyca/fides/compare/2.39.0...2.39.1)

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

### Fixed
- Fixed a bug where system information form was not loading for Viewer users [#5034](https://github.com/ethyca/fides/pull/5034)
- Fixed viewers being given the option to delete systems [#5035](https://github.com/ethyca/fides/pull/5035)
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 ceb1c60

Please sign in to comment.