Making WordPress.org

Opened 10 years ago

Closed 7 years ago

#720 closed enhancement (fixed)

Notify sponsors whether venues have exhibition space or not

Reported by: iandunn's profile iandunn Owned by: iandunn's profile iandunn
Milestone: Priority: normal
Component: WordCamp Site & Plugins Keywords: good-first-bug needs-testing
Cc:

Description

When sponsors are sent automated e-mails about new WordCamps, they should be notified whether or not the venue has exhibition space.

To do this, we need to:

  1. Add a checkbox to the WordCamp post type
  2. Add a placeholder shortcode to Organizer Reminders plugin

Attachments (2)

patch_720_meta.diff (3.1 KB) - added by bansod_deven 10 years ago.
Patch for #720 Meta
patch_720_meta.2.diff (1.1 KB) - added by iandunn 10 years ago.
Refresh after r1232

Download all attachments as: .zip

Change History (7)

#1 @couturefreak
10 years ago

  • Cc oxocodes@… added

#2 @bansod_deven
10 years ago

  • Keywords dev-feedback needs-testing added

Added a Patch.

  1. Added a 'Is Exhibition Space available ?' Checkbox to WordCamp Post Type in the Venue Section.
  1. Placeholder Shortcode
    ['venue_has_exhibition_space']
    
    added in Organizers Reminders Plugin.

Needs Testing though I could not test if the mails are going as expected or not.

Last edited 10 years ago by bansod_deven (previous) (diff)

@bansod_deven
10 years ago

Patch for #720 Meta

#3 @iandunn
10 years ago

In 1232:

WordCamp Post Type: Add a field to track if venues have exhibition space.

Props bansod_deven
See #720

#4 @iandunn
10 years ago

  • Keywords dev-feedback removed
  • Owner set to iandunn
  • Status changed from new to accepted

This looks good, thanks :)

I committed the first part in r1232. One small change I made was to use checked() instead of manually outputting the attribute.

I haven't committed the other part yet, because we now also want to tie the new shortcode to sponsorship packages, because only certain packages include exhibition space, so only those sponsors should have the information included in their email.

The packages are currently being updated, though, so we'll need to wait until they're settled in order to know what criteria we can use to programmatically determine whether or not to include the message about space being available. That may be done later today, though.

Once that's added, the second part just needs a final review/test and then it can be merged.

@iandunn
10 years ago

Refresh after r1232

#5 @iandunn
7 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 5356:

WordCamp Organizer Reminders: Add the venue_exhibition_space_message shortcode

Props bansod_deven
Fixes #720

Note: See TracTickets for help on using tickets.