-
-
docs/codebase/extending/extensions.txt (Diff revision 1) To reference a file, you can use :file:`extension.py`.
-
docs/codebase/extending/extensions.txt (Diff revision 1) Instead of this, an alternative is to create: self.reviewui_hooks = ReviewUIHook(self, [XMLReviewUI]) in the extension's __init__. I think that's a better model in general for us to document, as it's less code and achieves the same thing. That can then more easily be merged into the code below.
-
-
docs/codebase/extending/extensions.txt (Diff revision 1) While nice to demonstrate, we should keep this as simple as possible to follow and not make people wonder if they need these or what they're for.
-
docs/codebase/extending/extensions.txt (Diff revision 1) I'm actually not familiar with this format. What are we doing?
-
-
-
-
-
-
docs/codebase/extending/extensions.txt (Diff revision 1) I think this should just be a :file:`...`. You won't get the bold, but I don't think that's a problem.
-
docs/codebase/extending/extensions.txt (Diff revision 1) Can you put a blank line around each block? Helps with readability.
-
docs/codebase/extending/extensions.txt (Diff revision 1) We don't use MEDIA_URL or MEDIA_SERIAL anymore. It should all be {% static %}, in theory. Have you tried that with your extension?
-
-
-
Documentation: How to Integrate ReviewUIs via Extensions
Review Request #3596 — Created Nov. 30, 2012 and submitted
Information | |
---|---|
slchen | |
Review Board | |
master | |
Reviewers | |
reviewboard | |
A how-to guide for creating and integrating ReviewUIs through ReviewBoard Extensions. Content adds to http://www.reviewboard.org/docs/codebase/dev/extending/extensions/
Ran `make html` in reviewboard/docs/codebase/ No error output from sphinx. Rendered HTML pass visual inspection.
Description | From | Last Updated |
---|---|---|
To reference a file, you can use :file:`extension.py`. |
|
|
Instead of this, an alternative is to create: self.reviewui_hooks = ReviewUIHook(self, [XMLReviewUI]) in the extension's __init__. I think that's a … |
|
|
Blank line after this. |
|
|
While nice to demonstrate, we should keep this as simple as possible to follow and not make people wonder if … |
|
|
Two blank lines. |
|
|
"syntax highlighted" |
|
|
Blank line between these. |
|
|
And here. |
|
|
And here. |
|
|
I think this should just be a :file:`...`. You won't get the bold, but I don't think that's a problem. |
|
|
Can you put a blank line around each block? Helps with readability. |
|
|
We don't use MEDIA_URL or MEDIA_SERIAL anymore. It should all be {% static %}, in theory. Have you tried that … |
|
|
:file: |
|
|
:file: |
|
|
Blank line between these. |
|