Bug 1819033

Summary:	Combine JS files as much as possible
Product:	[Community] Bugzilla	Reporter:	Jeff Fearn 🐞 <jfearn>
Component:	User Interface	Assignee:	Jeff Fearn 🐞 <jfearn>
Status:	CLOSED CURRENTRELEASE	QA Contact:	tools-bugs <tools-bugs>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5.0	CC:	khong, meli
Target Milestone:	5.0-RH11
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	5.0.4-rh41	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-04-06 00:15:59 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jeff Fearn 🐞 2020-03-31 02:21:00 UTC

Description of problem:
Currently there are a lot of JavaScript files, this can cause a problem when users are automatically refreshing a lot of tabs.

Version-Release number of selected component (if applicable):
5.0

How reproducible:
Easy

Steps to Reproduce:
1. Load a page
2. refresh page

Actual results:
Up to 29 JS files checked

Expected results:
<= 6 JS files checked

Additional info:
6 is a guess, it may not be possible to go this low.

The technique used to consolidate the files can break some JS, so extensive testing may be required.

This can be done by pushing JS files on to javascript_urls array.

Comment 1 Jeff Fearn 🐞 2020-04-01 01:26:20 UTC

QE Note: This reduces the number of JS files delivered to the user to 3. Concatenating the JS files this way could have unforeseen effects in the UI, this means every JS based feature needs to be tested logged in, logged out, and after logging in or out.

Comment 2 Meiying Li 2020-04-03 06:13:50 UTC

Verified it because there is no regression from the automation result.
https://redhat-bugzilla-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/bz-web-automaiton/97
https://redhat-bugzilla-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/bz-web-automaiton/98

Comment 4 Jeff Fearn 🐞 2020-04-06 00:15:59 UTC

This change is now live. If there are any issues, do not reopen this bug. Instead, you should create a new bug and reference this bug.