Skip to content

Add crossorigin script attribute for local development Webpack assets#6126

Merged
aduth merged 4 commits intomainfrom
aduth-fix-react-dev-errors
Mar 31, 2022
Merged

Add crossorigin script attribute for local development Webpack assets#6126
aduth merged 4 commits intomainfrom
aduth-fix-react-dev-errors

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Mar 30, 2022

Why: To prevent issues where React is unable to access error details in development mode, in combination with assets served cross-origin via webpack-dev-server.

Steps to reproduce: Follow "Testing identity proofing" process to upload an error document to the document capture step. Without these changes, you would see a generic "technical difficulties" message.

Related docs: https://reactjs.org/docs/cross-origin-errors.html

**Why**: To prevent issues where React is unable to access error details in development mode, in combination with assets served via webpack-dev-server.
@aduth aduth changed the title Add crossorigin tag for local development Webpack assets Add crossorigin script attribute for local development Webpack assets Mar 30, 2022
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

aduth and others added 3 commits March 30, 2022 18:40
https: //github.com//pull/6126/files#r839008756
Co-Authored-By: Zach Margolis <zbmargolis@gmail.com>
changelog: Internal, Local Development, Fix cross-origin React errors when developing locally
@aduth aduth merged commit 164231d into main Mar 31, 2022
@aduth aduth deleted the aduth-fix-react-dev-errors branch March 31, 2022 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants