There needs to be an easy way to connect to Apps servers via the admin.

CommentFileSizeAuthor
#1 apps-connector-1412666-2.patch14.28 KBTomDude48
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TomDude48’s picture

Title: UI for adding Apps server connections » apps_connect sub module
Status: Active » Needs review
FileSize
14.28 KB

Attached is the apps_connector sub module we worked on Friday. I have cleaned it up and added a couple of thing like a directory of Apps servers you can enable with one click. I also got the integration into the main apps admin setting page done.

e2thex’s picture

Ok three things here.

1. I do not think we should prepopulate with app servers (even if they are not active). I think this would cause issue deciding which get in and which do not. (it seems resonable for some one to create a module that does this but not part of the apps module.

2. I think apps_connector_server_listings_info is redundent with apps_server_info. But i love the fact that there is an option to activate and deactivate them. I think use the main hook but allowing disabiling would be lovely.

3. Which feeds in to number 3, after looking atht he admin include it seems clear that we are doing alot of stuff that ctools exports gets us. and we might want to change this into a ctools exportable.

This might make this whole add on very small (and then maybe we could make the dev server just a export of that that starts disabled

e2thex’s picture

Status: Needs review » Needs work