Good idea and implementation.
My first [small] stumbling block after installing the module was in figuring out the green checkmark and red X's on the Network Display screen.
An explicit explanation that Network Manager :: Network Connector should be installed on managed sites would be nice.
I see the tooltip now, but something more apparent would help :)
But actually...
if all that Network Connector provides is a way to add links to the sites and their functionality from the management interface, those icons might be overkill. The red X communicates an error exists, yet the tool is still entirely useful and working without the links that the Connector provides -- and without installing another module on all other sites...
What if the hostname/basepath for managed sites could be manually set (per site) within Network Manager? Set manually or install the Connector for automatic discovery... I'd prefer to make the management module a bit heavier than to have to install [and manage] an extra module on every other site.
Comments
Comment #1
amariotti commentedI second all that he said above. It appears that the only added feature is turning the Network Overview page into linkable items. If there's more than point it out. If the Network Connector is not necessary then it might be a good idea to pull it out. Thoughts, Bryan!?
Comment #2
btopro commentedThe network connector was put in place to allow for future, extensible functionality to actually bridge the sites together so that a site running manager (with proper authorization) could connect to and change settings on a network connector based site. As that might be a bit overkill in scope for the time being I think it makes sense to drop network connector and the icons to force people to have to input the address on the network manager site. I'll investigate the implications of doing that now...
Comment #3
btopro commentedwill be rolled into version 6.x-1.3