Bug 1518547 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Here's a list of each change and my verification of it. Don't think we can deploy this to prod as it is.



bug 1514765 : New bug comment preview broken when use_markdown is disabled
  - VERIFIED
bug 1515144 : Prevent Google Analytics script from being loaded when DNT:1 so that Tracking Protection doesn't have to block it
  - Not able to verify
bug 1515405 : Add rate limiting for comments and attachment accesses
  - ❗ Broken? - Not able to load image or github attachments (since it talks to bmo first) anymore on staging. See comment above.
bug 1514848 : Add utility to revoke API keys
  - Not able to verify
bug 1506725 : Update form.doc to direct reporters to GitHub
  - Likey Ok, but, I'm not sure where this page is to verify
bug 1493253 : Embed crash count table to bug pages
  - ❗ Broken? - Crash Data section empty on https://bugzilla.allizom.org/show_bug.cgi?id=1463048
bug 1514446 : Disabled users don't have 'strikethrough' in group owners report
  - VERIFIED
bug 1508389 : Change links for WebCompat reporting
  - VERIFIED
bug 1516021 : Rename all references in the code and file tree from Quantum to App
  - VERIFIED
bug 1514411 : NSS, DevTools and other products don't have "Has STR" and "Has Regression Range" fields
  - VERIFIED
bug 1516188 : Alignment of "Bugs Filed" section in profile page is not same as of rest of rows below it.
  - VERFIED
bug 1517606 : Infinite loop in Bugzilla::Extension::BMO::_inject_headers_into_body()
  - Not able to verify
bug 1518381 : lightbox is no longer used when clicking on an attachment's link in markdown comments
  - VERIFIED
bug 1512526 : Revisions table: mouse over anywhere in the reviewer name should pop up the review status
bug 1509122 : Need additional space between status icon and status text in the revision panel in show_bug
  - Not able to verify (bugzilla-staging has no connected phabricator instance)
bug 1518264 : New non-monospace comments styled with way too small a font size
  - VERIFIED (someone else might want to check that it looks good on their screen too)
bug 1518380 : reduce the margins of blockquotes in html comments
  - VERIFIED
bug 1518350 : Allow literal html tags to stand for themselves
  - VERIFIED
bug 1518303 : Comments on splinter patches are misformatted with markdown styling
  - VERIFIED
bug 1518218 : The attachment icon is misplaced
  - VERIFIED (it was removed, the patch to align it will be in next deploy!)
bug 1518268 : Preview cuts off bottoms of things using ```
  - VERIFIED
bug 1518266 : Font of comment emails is rendered too large since markdown has been enabled
  - Not able to verify
Here's a list of each change and my verification of it. Don't think we can deploy this to prod as it is.



- bug 1514765 : New bug comment preview broken when use_markdown is disabled
  - VERIFIED
- bug 1515144 : Prevent Google Analytics script from being loaded when DNT:1 so that Tracking Protection doesn't have to block it
  - Not able to verify
- bug 1515405 : Add rate limiting for comments and attachment accesses
  - ❗ Broken? - Not able to load image or github attachments (since it talks to bmo first) anymore on staging. See comment above.
- bug 1514848 : Add utility to revoke API keys
  - Not able to verify
- bug 1506725 : Update form.doc to direct reporters to GitHub
  - Likey Ok, but, I'm not sure where this page is to verify
- bug 1493253 : Embed crash count table to bug pages
  - ❗ Broken? - Crash Data section empty on https://bugzilla.allizom.org/show_bug.cgi?id=1463048
- bug 1514446 : Disabled users don't have 'strikethrough' in group owners report
  - VERIFIED
- bug 1508389 : Change links for WebCompat reporting
  - VERIFIED
- bug 1516021 : Rename all references in the code and file tree from Quantum to App
  - VERIFIED
- bug 1514411 : NSS, DevTools and other products don't have "Has STR" and "Has Regression Range" fields
  - VERIFIED
- bug 1516188 : Alignment of "Bugs Filed" section in profile page is not same as of rest of rows below it.
  - VERFIED
- bug 1517606 : Infinite loop in Bugzilla::Extension::BMO::_inject_headers_into_body()
  - Not able to verify
- bug 1518381 : lightbox is no longer used when clicking on an attachment's link in markdown comments
  - VERIFIED
- bug 1512526 : Revisions table: mouse over anywhere in the reviewer name should pop up the review status
- bug 1509122 : Need additional space between status icon and status text in the revision panel in show_bug
  - Not able to verify (bugzilla-staging has no connected phabricator instance)
- bug 1518264 : New non-monospace comments styled with way too small a font size
  - VERIFIED (someone else might want to check that it looks good on their screen too)
- bug 1518380 : reduce the margins of blockquotes in html comments
  - VERIFIED
- bug 1518350 : Allow literal html tags to stand for themselves
  - VERIFIED
- bug 1518303 : Comments on splinter patches are misformatted with markdown styling
  - VERIFIED
- bug 1518218 : The attachment icon is misplaced
  - VERIFIED (it was removed, the patch to align it will be in next deploy!)
- bug 1518268 : Preview cuts off bottoms of things using ```
  - VERIFIED
- bug 1518266 : Font of comment emails is rendered too large since markdown has been enabled
  - Not able to verify

Back to Bug 1518547 Comment 3