Integration forms and config template

Review Request #7136 - Created March 30, 2015 and discarded

Xuanyi Lin
Review Board
master
6918, 7105, 7106
7221
reviewboard, students

This provides the integration forms class to manage the configuration of a integration in the configure page. The config template allows integration to display additional information in the configure integration page.

This form provides some preset form fields that are common for an integration. Integration could further customize the form for its own configuration by subclassing the integration forms.

The config template allows integration to provide a more detailed description, with extra content to simplify the process of setting up an integration.

Manual testing with a dummy integration.
Pass all unit test.

Loading file attachments...

  • 8
  • 0
  • 6
  • 0
  • 14
Description From Last Updated
This should use super David Trowbridge David Trowbridge
This can be combined: try: return self.config.integration.default_configurations[key] except KeyError: raise KeyError(...) David Trowbridge David Trowbridge
Add another blank line here. David Trowbridge David Trowbridge
Add a trailing comma here. David Trowbridge David Trowbridge
Only one blank line here. David Trowbridge David Trowbridge
HTML should be single-space indented. David Trowbridge David Trowbridge
This should still be in a blocktrans. You can use blocktrans with to define the variable. David Trowbridge David Trowbridge
This should still be in a blocktrans. You can use blocktrans with to define the variable. David Trowbridge David Trowbridge
Review Bot
Jessica Yuen
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
David Trowbridge
Xuanyi Lin
Review request changed

Status: Discarded

Change Summary:

Closed in favor of a revised version of Integrations that landed for 3.0.

Loading...