Opened 9 years ago
Closed 6 years ago
#2383 closed enhancement (fixed)
Add a meta field in the Sponsor CPT for Twitter handle
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Component: | WordCamp Site & Plugins | Keywords: | good-first-bug has-patch needs-testing |
| Cc: |
Description (last modified by )
It would be a nice convenience for organizers if we had a field for a sponsor's twitter handle, so it’s easier to tweet a thank you. This would be particularly helpful in the case of global sponsors, but I could see a use case for local sponsors as well. It would not need to be a required field.
Update: the Twitter Handle field has been added for the Sponsor CPT. A similar field still needs to be added to the multi-event-sponsor plugin, though. See #3.
Attachments (3)
Change History (14)
#1
@
9 years ago
- Keywords good-first-bug needs-patch added
- Owner set to iandunn
- Status changed from new to accepted
#3
@
9 years ago
That looks good, thanks!
Eventually we'll also want to add a corresponding field to the multi-event-sponsors plugin, and copy the value from those global sponsors to the wcpt sponsor posts when new sites get created. 2383.diff can be committed without that, though.
We've got a backlog of patches to review and commit, but I've added this one to the list. Hoping to find some time soonish.
#6
@
9 years ago
- Keywords has-patch added; needs-patch removed
Hi @coreymckrill
I've added Twitter Handle: meta field in the multi-event-sponsors plugin. I've attached a patch. 2383.1.diff
Please check and let me know if it's fine.
Thanks,
#7
@
9 years ago
Hi @coreymckrill
I've also made changes in wcpt plugin to copy value of Twitter Handle: meta field from global sponsors to local sponsors on site clone as @iandunn suggested. But I'm not able to test this feature in my local environment.
I've attached patch. 2383.2.diff
HI @iandunn
I've added
Twitter Handle:meta field in Sponsor CPT. I've attached a patch. 2383.diffPlease check and let me know if it's fine.
Thanks,