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

CommentFileSizeAuthor
#1 apps-connector-1412666-2.patch14.28 KBTomDude48

Comments

TomDude48’s picture

TomDude48 commented
Title: UI for adding Apps server connections » apps_connect sub module
Status: Active » Needs review
StatusFileSize
new apps-connector-1412666-2.patch14.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

e2thex commented

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

e2thex commented
Status: Needs review » Needs work