Description of problem: SSUI has MIQ productization in various places instead of RH productization See screenshots. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info: Login screen, Header bar, about modal. Maybe more I don't know what all get's changed.
Created attachment 1280244 [details] login screen
Created attachment 1280245 [details] Header bar
Version 5.8.0.15
Just looking at the bg-login.png file (that's a good one to look at) it seems that all instances of that file (there are 3) are correct. However, looking at where webpack puts things, there is a file that is the incorrect image. This leads me to think that yard build is getting run before the productization files are in place.
Issue is here: https://github.com/ManageIQ/manageiq-ui-service/blob/105db14ae4f50df057b5c41c2d834861661df74b/config/webpack.dev.js#L99 If the file is small, and not present before we build, it doesn’t matter what’s in the skin folder, we serve a small URI encoded file. PR incoming.
sooooooo after a long day of bagging the head against the keyboard… our core issue with skinning is this. Some assets work and some assets don’t because the assets declarations are different. Declarations from the html with `src` refer to the images folder, which is fine because at the tail end of webpacking that is populated (in miq/public) with the desired skinned contents. Declarations from the sass files however get processed, which happens *before* the skinned assets are copied (but also, these assets look for images in the `client/app/assets/images` folder rather than the `/client/skinned` folder) SO I propose that we reconfigure the symlink to link the `sample-skin/images` folder to the ‘client/app/assets/images` folder in addition to `client/skinned`this will ensure compiled assets used skinned versions.
@Satoe - Thoughts on Allen's idea (Comment 8)?
Works for me - once needed change is done on SUI side (change to link.sh?), please assign this BZ to me so I can ensure downstream build makes use of the change.
https://github.com/ManageIQ/manageiq-ui-service/pull/790 Satoe, good news, you got nothing to do here, nothing changed with link.sh, essentially reconfigured webpack so that we're now dynamically setting asset urls in sass based on deployment environment, so when we're in production, we'll use the appropriate /images folder assets
Verified in 5.9.0.2
*** Bug 1501043 has been marked as a duplicate of this bug. ***