Add integration model and manager for third-party services framework

Review Request #6918 - Created Feb. 7, 2015 and discarded

Xuanyi Lin
Review Board
master
7221, 7105, 7136, 7106, 7034, 7033
reviewboard, students

This provides the integration base class, database model and manager for extensions to implement and register integrations.

Compared to the extension framework, these integration model will provide a way for users, such as local site administrator, to manage their own individiual integration. Third-party services will need to subclass the integration base class to provide the initialize and shutdown method for their services.

With the database model, multiple instances of the integration's configuration could be supported, and thus providing multiple configurations of the integration for a single web server.

All these configured integration will then be managed through the integration manager.

Added unit test for the integrations.

  • Test manager on creating, deleting, toggling of configured integration.
  • Test manager on handling of invalid configured integration.

Mannual testing with a dummy integration.

Unit tests pass.

  • 15
  • 0
  • 74
  • 53
  • 142
Description From Last Updated
Imports should come in alphabetical order. David Trowbridge David Trowbridge
Should be in the imperative mood ("Wrap a callback ...") David Trowbridge David Trowbridge
exc_info should be True, rather than 1 (we have 1 a bunch of other places, but I'm slowly cleaning those ... David Trowbridge David Trowbridge
This should use the same terminology as you changed above ("registration" rather than "initializing") David Trowbridge David Trowbridge
This should use the same terminology as you changed above ("registration" rather than "initializing") David Trowbridge David Trowbridge
Should be in the imperative mood ("Shut down...") David Trowbridge David Trowbridge
Should be in the imperative mood ("Return the...") David Trowbridge David Trowbridge
Should be in the imperative mood ("Return the...") David Trowbridge David Trowbridge
Should be in the imperative mood ("Return the...") David Trowbridge David Trowbridge
The logging methods will do a format operation internally, so you can pass in config_id as a parameter rather than ... David Trowbridge David Trowbridge
Should be in the imperative mood ("Return all...") David Trowbridge David Trowbridge
Should be in the imperative mood ("Return the...") David Trowbridge David Trowbridge
Should be in the imperative mood ("Return the...") David Trowbridge David Trowbridge
Should be in the imperative mood ("Return...") David Trowbridge David Trowbridge
Should be in the imperative mood ("Return...") David Trowbridge David Trowbridge
Review Bot
Xuanyi Lin
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Christian Hammond
Barret Rennie
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Xuanyi Lin
Review Bot
Christian Hammond
Christian Hammond
Xuanyi Lin
Xuanyi Lin
Review Bot
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...