fix IE11 error from sentry
What does this MR do?
remove unsupported IE11 URL()
constructor
Are there points in the code the reviewer needs to double check?
- there are a few other
new URL
s in PDF stuff, should they be fixed as well? - should we lint for this? Couldn't find an existing one so might need modified http://eslint.org/docs/rules/no-new-wrappers
Why was this MR needed?
bug https://sentry.gitlap.com/gitlab/gitlabcom-clientside/issues/30928/
Fixed in edge - https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/3775534/
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary [ ] Documentation created/updated[ ] API support added- Tests
-
Added for this feature/bug -
All builds are passing
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Branch has no merge conflicts with master
(if it does - rebase it please) -
Squashed related commits together
What are the relevant issue numbers?
Closes #32538 (closed)
Edited by Simon Knox
Merge request reports
Activity
@mikegreiling can you please review? small fix for sentry error
Edited by Simon Knoxassigned to @mikegreiling
LGTM. @timzallmann can you merge this?
assigned to @timzallmann
mentioned in commit fc56d2fb
Please register or sign in to reply