As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 1237185 - hide 'cab review' custom field behind a "click through" to direct people to servicenow
: hide 'cab review' custom field behind a "click through" to direct people to s...
Classification: Other
Component: General (show other bugs)
: Production
: Unspecified Unspecified
: -- normal (vote)
: ---
Assigned To: Byron Jones ‹:glob›
Depends on:
  Show dependency treegraph
Reported: 2016-01-06 00:22 PST by Byron Jones ‹:glob›
Modified: 2016-01-11 07:01 PST (History)
2 users (show)
See Also:
Due Date:
QA Whiteboard:
Iteration: ---
Points: ---

Screenshot 2016-01-06 12.38.15.png (10.47 KB, image/png)
2016-01-06 00:22 PST, Byron Jones ‹:glob›
no flags Details
1237185_1.patch (5.72 KB, patch)
2016-01-06 19:27 PST, Byron Jones ‹:glob›
no flags Details | Diff | Splinter Review

Description User image Byron Jones ‹:glob› 2016-01-06 00:22:26 PST
Created attachment 8704513 [details]
Screenshot 2016-01-06 12.38.15.png

tracking of cab changes is moving from bmo to servicenow early/mid next week (11th jan).

jason contacted me asking what we can do to ease that migration, such as adding a link to the servicenow form from show_bug.

to ease migration jason requested that for another month the cab-review custom field still be functional on bmo, but we'll make it harder to set by hiding it behind a click that people have to 'close' before they can set the field (see attached mock-up).

jason - can you provide the exact servicenow url we need to link to?
Comment 2 User image Byron Jones ‹:glob› 2016-01-06 19:27:57 PST
Created attachment 8704941 [details] [diff] [review]

- hides cab-review field behind a link to servicenow
- only happens on bugs where the cab-review field is not set
Comment 3 User image Byron Jones ‹:glob› 2016-01-10 19:38:00 PST
Comment on attachment 8704941 [details] [diff] [review]

as per irc, this needs to land today.
<dylan> I looked at the patch and I'm pretty sure it's an r+
landing as-is, can fix issues in a follow-up bug.
Comment 4 User image Byron Jones ‹:glob› 2016-01-10 20:28:05 PST
To ssh://
   9bc226a..9e478dd  master -> master
Comment 5 User image Byron Jones ‹:glob› 2016-01-11 07:01:03 PST
oops, this broke js on the old ui:

To ssh://
   9e478dd..4b60281  master -> master

Note You need to log in before you can comment on or make changes to this bug.