-
Notifications
You must be signed in to change notification settings - Fork 230
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
[3.6] Fix Bug 1543435 - A-MQ Components not visible in Java Console when using IE11 browser #2788
[3.6] Fix Bug 1543435 - A-MQ Components not visible in Java Console when using IE11 browser #2788
Conversation
/ok-to-test |
@akieling Unfortunately, you might need to use an older version of node in our older release branches. We had an issue where you would generate different dist depending on the version you used. Verifying that checked in built files under dist match the source...
diff --git a/dist/scripts/vendor.js b/dist/scripts/vendor.js
index 1057b572..d27002ff 100644
--- a/dist/scripts/vendor.js
+++ b/dist/scripts/vendor.js
@@ -8062,7 +8062,7 @@ delete c[b.id], delete d[a];
http:80,
https:443,
ftp:21
-}, sf = b("$location"), tf = /^\s*[\\\/]{2,}/, uf = {
+}, sf = b("$location"), tf = /^\s*[\\/]{2,}/, uf = {
$$absUrl:"",
$$html5:!1,
$$replace:!1, |
Related issue for the dist problem: #1667 This is fixed in master, but not for older branches. You can either install a node version older than v4 or manually undo this change in vendor.js |
@spadgett I've rebuilt the project using npm 3.10.10 and it seems to have fixed the issue with vendor.js. |
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.
/lgtm
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. |
Fixes the following bug on OpenShift Enterprise 3.6:
https://bugzilla.redhat.com/show_bug.cgi?id=1543435