Review Request #5769 — Created May 4, 2014 and discarded
Add support for bug infobox on Bugzilla
Tested against a local Bugzilla instance and Bugzilla demo at https://bugzilla.mozilla.org/
| Description | From | Last Updated |
|---|---|---|
|
Please add a docstring. |
david
|
|
|
These can be combined into one statement. |
david
|
|
|
Please add a docstring. |
david
|
|
|
These can be combined into one statement. |
david
|
|
|
Can you reformat this to put the dictionary entries on their own lines? (newline after { and before }). |
david
|
|
|
If there's an exception, this will return None. Should it return '' instead? |
david
|
|
|
Same question about None vs ''. |
david
|
Add caching, adapt to changes in r/5531 and resolve Bugzilla url from repository
Revision 2 (+43 -1)
Add support for bug infoboz on Bugzilla | ||
| - | ||
| - | TODO: | |
| - | Define how to get the bug tracker url on BugTracker | |
| - | Caching |
| ~ | Add support for bug infoboz on Bugzilla | |
| ~ | Add support for bug infobox on Bugzilla |
Take out of WIP
Fix issues
Revision 3 (+43 -1)
Add Bugzilla url as part of the cache key
Revision 4 (+45 -1)
Add requested docstrings
Revision 5 (+47 -1)
Fix issues
Revision 6 (+49 -1)
