WordPress.org

Making WordPress.org

Opened 5 years ago

Closed 2 years ago

#720 closed enhancement (fixed)

Notify sponsors whether venues have exhibition space or not

Reported by: iandunn Owned by: 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 5 years ago.
Patch for #720 Meta
patch_720_meta.2.diff (1.1 KB) - added by iandunn 5 years ago.
Refresh after r1232

Download all attachments as: .zip

Change History (7)

#1 @couturefreak
5 years ago

  • Cc oxocodes@… added

#2 @bansod_deven
5 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.

Version 0, edited 5 years ago by bansod_deven (next)

@bansod_deven
5 years ago

Patch for #720 Meta

#3 @iandunn
5 years ago

In 1232:

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

Props bansod_deven
See #720

#4 @iandunn
5 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
5 years ago

Refresh after r1232

#5 @iandunn
2 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.