Details
-
Improvement
-
Resolution: Not an issue
-
Minor
-
None
-
1.0
-
None
Description
It would be a nice improvement if we could alert the user that the configuration credentials must be set in order for the module to start correctly.

When you install the module the first time you will inevitably hit this issue:
2016-07-07 18:37:47,811 INFO info.magnolia.module.ModuleManagerImpl : Initializing module google-analytics 2016-07-07 18:37:47,895 INFO info.magnolia.module.ModuleManagerImpl : Initializing module google-analytics-visualization 2016-07-07 18:37:47,909 INFO info.magnolia.module.ModuleManagerImpl : Starting module google-analytics-visualization 2016-07-07 18:37:47,909 ERROR lia.ga.visualization.GoogleAnalyticsServiceFactory: Can't initialize GoogleCredential. Mandatory fields are missing in google analytics credential config. Please fix your configuration under [config:/modules/google-analytics-visualization/credentialConfig]. 2016-07-07 18:37:47,925 WARN ent.registry.GoogleAnalyticsClientObservingManager: Can't schedule clients because google analytics service isn't initialized properly, please check credential configuration.
Unless you bootstrap in the credentials before the module starts, which is unlikely for someone testing out the module.
When you try to open the app in crashes because the module was never able to start. The app can't know about the issue that occurred during startup so it crashes with a somewhat generic Guice provision error.
So when the module installs it would be helpful to add a message about setting the configuration.
Checklists
Acceptance criteria