Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run playground Gen1/2 in IE11 #3599

Merged
merged 4 commits into from
Apr 17, 2024
Merged

Conversation

denysoblohin-okta
Copy link
Contributor

@denysoblohin-okta denysoblohin-okta commented Apr 10, 2024

Description:

Modified webpack configs to use env var IE11_COMPAT_MODE for running playground Gen2 in IE11

Usage:

yarn start:ie11

PR Checklist

Issue:

Reviewers:

Screenshot/Video:

Downstream Monolith Build:

trevoring-okta
trevoring-okta previously approved these changes Apr 10, 2024
Copy link
Contributor

@lesterchoi-okta lesterchoi-okta left a comment

Choose a reason for hiding this comment

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

Could you add a yarn:start:ie11 script to package.json?

'@babel/preset-typescript' // must run before preset-env: https://github.com/babel/babel/issues/12066
]
}
},
]
},
devServer: {
...hotReloadOptions,
host: HOST,
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you also set the host option of the webpack-dev-server to 0.0.0.0 in IE11_COMPAT_MODE since it's common to want to test in a local VM?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added

"start:ie11": "IE11_COMPAT_MODE=true OKTA_SIW_HOST=0.0.0.0 ENTRY=default yarn start --watch",

@denysoblohin-okta denysoblohin-okta changed the title Run playground Gen2 in IE11 Run playground Gen1/2 in IE11 Apr 12, 2024
@oktapp-aperture-okta oktapp-aperture-okta bot merged commit f999c2b into master Apr 17, 2024
1 check passed
@oktapp-aperture-okta oktapp-aperture-okta bot deleted the od-ie11-playground-gen2 branch April 17, 2024 10:41
trevoring-okta pushed a commit that referenced this pull request Apr 23, 2024
OKTA-714198 Adds script `start:ie11`
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.

None yet

3 participants