Uploaded image for project: 'Google Analytics Visualization'
  1. Google Analytics Visualization
  2. MGNLGAV-30

Installation should alert the user to set credentials

    XMLWordPrintable

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

        Attachments

          Activity

            People

              Unassigned Unassigned
              rgange Richard Gange
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Task DoD