Adding documentation for AdminWidgetHook
Review Request #6597 — Created Nov. 16, 2014 and submitted
Information | |
---|---|
justy777 | |
Review Board | |
master | |
|
|
f077343... | |
Reviewers | |
reviewboard, students | |
Provides documentation and sample code for the newly created AdminWidgetHook.
Description | From | Last Updated |
---|---|---|
How about "allows extensions to register new widgets for the administration dashboard." |
|
|
Can you add the ugettext_lazy import to the list of imports so that _() is defined? |
|
|
I think we're targetting this work for 2.1. |
|
|
Make sure the trailing whitespace is removed. |
|
|
No blank line here, since from the extension's point of view, reviewboard is a third-party module. |
|
|
Two blank lines. |
|
-
-
docs/manual/extending/extensions/hooks/admin-widget-hook.rst (Diff revision 1) How about "allows extensions to register new widgets for the administration dashboard."
-
docs/manual/extending/extensions/hooks/admin-widget-hook.rst (Diff revision 1) Can you add the ugettext_lazy import to the list of imports so that
_()
is defined?
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+39) |

-
Tool: PEP8 Style Checker Ignored Files: docs/manual/extending/extensions/hooks/index.rst docs/manual/extending/extensions/hooks/admin-widget-hook.rst Tool: Pyflakes Ignored Files: docs/manual/extending/extensions/hooks/index.rst docs/manual/extending/extensions/hooks/admin-widget-hook.rst
-
-
docs/manual/extending/extensions/hooks/admin-widget-hook.rst (Diff revision 2) I think we're targetting this work for 2.1.
-
docs/manual/extending/extensions/hooks/admin-widget-hook.rst (Diff revision 2) Make sure the trailing whitespace is removed.
-
docs/manual/extending/extensions/hooks/admin-widget-hook.rst (Diff revision 2) No blank line here, since from the extension's point of view,
reviewboard
is a third-party module. -
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+39) |