Addressing comments and enhancing support
-Fixing Status Filter search
-Adding README.md to help future edits to the file
-Fixing the team filter for lower/upper case combinations
BUG=None
TEST=go/launchbugs?
- go/launchbugs?te=new
- go/launchbugs?m=105
- go/launchbugs?m=105&te=dhaddock
- go/launchbugs?team=corex
- go/launchbugs?team=corex&m=105
- go/launchbugs?&m=101&te=dhaddock&s=Fixed
- go/launchbugs?team=corex&m=101&te=sdantuluri&status=Fixed&c=OS>Systems>Update
Change-Id: I14a4e845cbe07ac5f8fff75ebd7e20efbd4090a7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crostestutils/+/3756204
Reviewed-by: David Haddock <dhaddock@chromium.org>
Tested-by: Wendy Aleman Martinez <awendy@chromium.org>
Reviewed-by: Wendy Aleman Martinez <awendy@chromium.org>
Commit-Queue: Wendy Aleman Martinez <awendy@chromium.org>
diff --git a/provingground/launchbugs/README.md b/provingground/launchbugs/README.md
new file mode 100644
index 0000000..1a182b5
--- /dev/null
+++ b/provingground/launchbugs/README.md
@@ -0,0 +1,40 @@
+# Launchbugs Redirect
+
+Launchbugs.html is a helper that opens filtered launchbugs based on Milestones, Test Engineers,
+Status, and component IIRC
+
+## Usage
+Navigate to [go/launchbugs?]
+Filter by Milestone: [go/launchbugs?m=105]
+Filter by Test Engineers: [go/launchbugs?te=dhaddock]
+Filter by Status: [go/launchbugs?m=105&s=Fixed]
+Filter by Component IIRC: [go/launchbugs?c=UI>Shell>StatusArea]
+Filter by Team: [go/launchbugs?team=coreX]
+
+## Filtering by Team
+Team is not an existing Filter in LaunchBugs. The functionality was added to shorten the Tests Engineers required if we want to add all the TEs from CoreX.
+Ex: `go/launchbugs?m=100&team=coreX` will search for all the Launch Bugs in Milestone 100 where the TEs are dhaddock,kyleshima,hyungtaekim, or awendy
+
+## Testing
+After modifying the script, please make sure that the old functionalities are still working, some good urls to test:
+- [go/launchbugs?]
+- [go/launchbugs?te=new]
+- [go/launchbugs?m=105]
+- [go/launchbugs?m=105&te=dhaddock]
+- [go/launchbugs?team=corex]
+- [go/launchbugs?team=corex&m=105]
+- [go/launchbugs?&m=101&te=dhaddock&s=Fixed]
+- [go/launchbugs?team=corex&m=101&te=sdantuluri&status=Fixed&c=OS>Systems>Update]
+
+[//]: # (Reference Links)
+[go/launchbugs?]: go/launchbugs?
+[go/launchbugs?te=new]: go/launchbugs?te=new
+[go/launchbugs?te=dhaddock]: go/launchbugs?te=dhaddock
+[go/launchbugs?m=105&s=Fixed]: go/launchbugs?m=105&s=Fixed
+[go/launchbugs?c=UI>Shell>StatusArea]: go/launchbugs?c=UI>Shell>StatusArea
+[go/launchbugs?m=105]: go/launchbugs?m=105
+[go/launchbugs?m=105&te=dhaddock]: go/launchbugs?m=105&te=dhaddock
+[go/launchbugs?team=corex]: go/launchbugs?team=corex
+[go/launchbugs?team=corex&m=105]: go/launchbugs?team=corex&m=105
+[go/launchbugs?&m=101&te=dhaddock&s=Fixed]: go/launchbugs?&m=101&te=dhaddock&s=Fixed
+[go/launchbugs?team=corex&m=101&te=sdantuluri&status=Fixed&c=OS>Systems>Update]: go/launchbugs?team=corex&m=101&te=sdantuluri&status=Fixed&c=OS>Systems>Update
diff --git a/provingground/launchbugs/launchbugs.html b/provingground/launchbugs/launchbugs.html
index 16dfb48..8523869 100644
--- a/provingground/launchbugs/launchbugs.html
+++ b/provingground/launchbugs/launchbugs.html
@@ -36,7 +36,7 @@
}
else {
console.log('replace status')
- src = src.replace(/<status>/g, "ChromeOS-Test:"+s)
+ src = src.replace(/<status>/g, "Status:"+s)
}
if (!c) {
console.log('No component')
@@ -52,6 +52,15 @@
var te = GetURLParameter('te')
var s = GetURLParameter('status')
var c = GetURLParameter('c')
+ //Add use of team variables
+ var team = GetURLParameter('team')
+ if (team && team.toLowerCase() == "corex"){
+ if (!te)
+ te = 'dhaddock,kyleshima,hyungtaekim,awendy'
+ else if(te != "new"){
+ te = te.concat(',dhaddock,kyleshima,hyungtaekim,awendy')
+ }
+ }
if (te == "new") {
console.log('te = new url')
window.location.replace('https://bugs.chromium.org/p/chromium/issues/list?sort=-chromeos-test&colspec=ID%20Component%20Summary%20TL%20PM%20TE%20ChromeOS-Test%20stable.m-target%20ChromeOS-Test-approver&num=100&q=Type%3DFLT-Launch%20OS%3DChrome%20-OS%3DIOS%2CAndroid%2CLinux%2CWindows%2CMac%2CFuchsia%20-ChromeOS-Test%3DAPPROVED%20has:ChromeOS-Leadership-Full%20-has:TE&can=2')
