WordPress.org

Making WordPress.org

Opened 5 years ago

Closed 5 years ago

#690 closed enhancement (fixed)

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

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

Description


Attachments (3)

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

Download all attachments as: .zip

Change History (8)

@nvwd
5 years ago

@kovshenin
5 years ago

#1 @kovshenin
5 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
5 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
5 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
5 years ago

#4 @nvwd
5 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
5 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.