Currently the message_subscribe_email feature is under the general 'features' tab by default. It should be moved into the Messages project.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amitaibu’s picture

Status: Needs review » Needs work

Actually message_subscribe_email should not be a "feature". I prefer to manually declare all the needed hooks. Any chance you can work on it?

ezra-g’s picture

Status: Needs work » Needs review
FileSize
770 bytes

Message subscribe already implements the necessary hooks. The only thing we need to do to prevent it from being a Features module is to remove the Features-specific entries from the .info file.

ezra-g’s picture

Title: Message Subscribe email should exist within other message features » Remove unnecessary Features lines from message_subscribe_info file
achton’s picture

Sorry, can't get #3 to apply against current -dev version.

patching file message_subscribe_email/message_subscribe_email.info
Hunk #2 FAILED at 7.
1 out of 2 hunks FAILED -- saving rejects to file message_subscribe_email/message_subscribe_email.info.rej
ezra-g’s picture

It will only apply to the module pulled from Git - the dev version is packaged with added lines in the .info that cause the patch to fail.

amitaibu’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.