Hi Ezra Gildesgame,
Lisa Rex created the event “Hackathon!” on Grupal Droups in the Engineering team group on December 17, 2012:

What: Hackathon
When: December 18, 2012 at 11:00 (EST)

Permalink: http://droups.grupal.org/2012/august/boston-food-truck-festival

Add a comment at http://droups.grupal.org/somethingsomethingsomething#comment-form
Change email notifications at http://droups.grupal.org/settings
CommentFileSizeAuthor
#2 1869006-events-message-template-2.patch12.63 KBjaperry

Comments

lisarex’s picture

lisarex commented
Issue summary: View changes

remove '

ezra-g’s picture

ezra-g commented
Title: New Event email notification » Proposed template for New Event email notification
Version: » 7.x-3.x-dev
Issue summary: View changes
Status: Active » Needs review
japerry’s picture

japerry
Location KOMK
commented
Issue tags: +commons 7.x-3.12 radar
StatusFileSize
new 1869006-events-message-template-2.patch12.63 KB

Here is a patch that adds the above suggested template.

devin carlson’s picture

devin carlson commented
Status: Needs review » Needs work

This worked great in my testing, I just saw a couple of minor issues.

  1. +++ b/modules/commons/commons_events/commons_events.install
@@ -223,3 +223,14 @@ function commons_events_update_7011() {
+function commons_events_update_7012() {
+  $revert = array(
+    'commons_events' => array('message_type'),
+  );
+  features_revert($revert);
+  return array();

    In my testing, the field_instance will also need to be reverted in order to accommodate the changes in commons_events.info or else Commons Events will show up as overridden on existing sites. 

  2. +++ b/modules/commons/commons_events/commons_events.module
@@ -601,6 +601,17 @@ function theme_commons_events_date_display_range_advanced($variables) {
 /**
+ * Implements commons_notify_message_selection().
+ */

    This should be:

    /**
 * Implements hook_commons_notify_message_selection_alter().
 */

  • Commit c462b9f on 7.x-3.x by japerry: 
    Issue #1869006 by japerry: Improved email template for event...
japerry’s picture

japerry
Location KOMK
commented
Status: Needs work » Fixed

Thanks Devin for the review, issues fixed in the commit!

  • Commit efddae5 on 7.x-3.x by japerry: 
    Issue #1869006 followup by japerry: Add commons_notify as a dependency...

Status: Fixed » Closed (fixed)

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