Hi guys,

While looking at several other modules, such as Google Analytics, or many other JS or Third Party related modules, I thought maybe we could consider prompting an error/warning on the Status report page if any CSNA Provide is installed but not configured correctly.
For example, if the Key or Secret configuration variables would not be configured yet, for the QQ Provider, then a message would be displayed in the Status report page to let users know about it.

Please let me know if you would have any questions, objections, comments, suggestions, recommendations or concerns on any aspects of this issue, I would be glad to provide more information or explain in more details.

Any questions, feedback, testing, changes, ideas or recommendations would be highly appreciated.
Thanks to all in advance.

CommentFileSizeAuthor
#1 Patch against csna-7.x-1.x branch at 9912c56 that should add implementation of hook_requirements in a new csna.install file.1.44 KBDYdave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DYdave’s picture

DYdave CreditAttribution: DYdave commented
Status: Active » Needs review
FileSize
Patch against csna-7.x-1.x branch at 9912c56 that should add implementation of hook_requirements in a new csna.install file.1.44 KB

Quick follow-up on this issue:

Please find attached to this comment a patch against csna-7.x-1.x branch at 9912c56 that should add implementation of hook_requirements in a new csna.install file.
File attached as: csna-requirements-validation-warning-wrong-provider-info-2074953-1.patch

The patch adds a new file called csna.install which contains the implementation of a standard API hook: hook_requirements that will display a warning message on the Status report page for each CSNA Provider that would not have been configured properly.

This patch has been tested and seems to work as expected, but I would greatly appreciate to have your feedback, questions, comments, reviews, suggestions, recommendations, improvements and testing for this patch.

Feel free to let me know if you would have any further comments, issues, questions, objections, recommendations, suggestions, testing, reporting or concerns on the attached patch or any other aspects of this ticket in general, I would be glad to provide more information or explain in further details.

Thanks in advance to all for your feedback, reviews, testing and reporting.
Cheers!