Making WordPress.org

Opened 10 years ago

Closed 10 years ago

#690 closed enhancement (fixed)

Allow Sponsor Wrangler to be selected within the Organizer Reminders admin page

Reported by: nvwd's profile nvwd Owned by: kovshenin's profile kovshenin
Milestone: Priority: normal
Component: WordCamp Site & Plugins Keywords: has-patch
Cc:

Description


Attachments (3)

690.patch (8.1 KB) - added by nvwd 10 years ago.
690-meta.diff (2.4 KB) - added by kovshenin 10 years ago.
690b-meta.diff (2.4 KB) - added by nvwd 10 years ago.

Download all attachments as: .zip

Change History (8)

@nvwd
10 years ago

@kovshenin
10 years ago

#1 @kovshenin
10 years ago

Hi @nvwd, thanks for your patch! I simplified it a bit in 690-meta.diff and removed some non-relevant changes like line breaks and whitespace. Can you please test it out before we deploy it to WordCamp.org?

#2 @iandunn
10 years ago

<label for="wcor_send_sponsor_wrangler">The Sponsor Wrangler (if set)</label>

That field is actually required now in order to change the post status to pending or draft, so it's probably safe to assume it'll exist.

#3 @nvwd
10 years ago

@kovshenin, thanks for compressing the diff, I have changed my IDE settings to stop messing with whitespace. I will test the patch this weekend and report back here.

@iandunn, thanks for the info, I will remove the '(if set)' text and upload another patch once I test.

@nvwd
10 years ago

#4 @nvwd
10 years ago

The form display works. Removed the '(if set)' from the diff.

I am not set up to test the execution of the reminder. @iandunn, would you be able to run a test please?

#5 @kovshenin
10 years ago

  • Owner set to kovshenin
  • Resolution set to fixed
  • Status changed from new to closed

In 1032:

WordCamp.org: Allow reminders to be sent to Sponsor Wranglers.

Fixes #690.
Props nvwd.

Note: See TracTickets for help on using tickets.