Having several mail modules in the system, it is nice to have them together.
- In modules page, it is not in the Mail module group.
- In modules page, there is no link to the configuration page.
- 'Forward' uses other config path settings as most other mail modules.
Both first issues can easily be changed in the .info file. A new path needs tweaking in other files, too.

--- forward.info	(revision 127)
+++ forward.info	(working copy)
@@ -7,6 +7,11 @@
 files[] = forward.views.inc
 files[] = forward.views_default.inc
 
+package = Mail
+configure = admin/config/user-interface/forward
+// todo: move config path to: 
+//configure = admin/config/system/forward

-

Comments

john.oltman’s picture

john.oltman commented
Assigned: Unassigned » john.oltman
Status: Active » Needs review

I added the modules page link to configuration. Leaving in user interface for now, as I see this module more as a sharing mechanism like FB Share than a mail transport module. The .info file change will be in tonight's DEV build. Thanks for mentioning this.

john.oltman’s picture

john.oltman commented
Status: Needs review » Closed (fixed)