-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Fix] [WRS-2276] Refactor studio loader config #281
base: main
Are you sure you want to change the base?
Conversation
Coverage report can be checked at https://stgrafxstudiodevpublic.blob.core.windows.net/studio-ui/coverage_reports/281/coverage.html |
Test Results185 tests 183 ✅ 2m 9s ⏱️ Results for commit 66fef17. ♻️ This comment has been updated with latest results. |
PR bundle size is 1.005 MB and 52.368 KB more than limit. |
PR bundle size is 1.005 MB and 52.364 KB more than limit. |
1 similar comment
PR bundle size is 1.005 MB and 52.364 KB more than limit. |
PR bundle size is 1.005 MB and 52.364 KB more than limit. |
PR bundle size is 1.005 MB and 52.335 KB more than limit. |
PR bundle size is 1.007 MB and 54.772 KB more than limit. |
PR bundle size is 1.007 MB and 54.747 KB more than limit. |
PR bundle size is 1.007 MB and 54.747 KB more than limit. |
PR bundle size is 1.007 MB and 54.747 KB more than limit. |
PR bundle size is 1.007 MB and 54.747 KB more than limit. |
PR bundle size is 1.011 MB and 59.079 KB more than limit. |
@@ -153,7 +153,7 @@ window.StudioUI.studioLoaderConfig({ | |||
projectName: projectName, | |||
|
|||
/* outputTypes: object of all available output types (optional) that have a boolean value */ | |||
outputTypes: outputTypes, | |||
outputSettings: outputSettings, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we could and should deprecate this, it hasn't any value anymore (output is not possible without output setting)
PR bundle size is 1.011 MB and 59.079 KB more than limit. |
This PR
Related tickets
Screenshots