Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#45036 closed enhancement (fixed)

Polldaddy is changing to Crowdsignal. Change oembed to suit.

Reported by: donncha's profile donncha Owned by: pento's profile pento
Milestone: 5.1 Priority: normal
Severity: normal Version: 5.1
Component: Embeds Keywords: has-patch needs-docs
Focuses: Cc:

Description

Polldaddy.com is changing to Crowdsignal.com in the next few weeks. The old oembed URL will still work but we're adding a new URL for surveys and using a new oembed URL.

polldaddy.com/oembed is becoming api.crowdsignal.com/oembed

The new oembed URL is active already.

Some test URLs to test the new oembed:

https://survey.fm/2438586

https://poll.fm/5yz3b

Attachments (1)

crowdsignal.diff (1.3 KB) - added by donncha 6 years ago.
Patch to change oembeds for Polldaddy.com to Crowdsignal.com

Download all attachments as: .zip

Change History (14)

@donncha
6 years ago

Patch to change oembeds for Polldaddy.com to Crowdsignal.com

#1 @SergeyBiryukov
6 years ago

  • Component changed from General to Embeds
  • Milestone changed from Awaiting Review to 5.0.1

#2 @pento
6 years ago

  • Milestone changed from 5.0.1 to 5.0.2

#3 @pento
6 years ago

  • Milestone changed from 5.0.2 to 5.0.3

#4 @audrasjb
6 years ago

  • Keywords dev-feedback added

Hi and thanks for the patch,

5.0.3 is going to be released soon. We are currently sorting the remaining tickets in the milestone.

The component maintainers for Embeds are @wonderboymusic @swissspidy and @azaozz

Do you think this ticket can be handled in the next couple of weeks? If not, let's address this in 5.1 which is coming soon.

#5 @swissspidy
6 years ago

  • Keywords dev-feedback removed
  • Owner set to swissspidy
  • Status changed from new to assigned

Should be an easy one. Just needs a docs update as well.

#6 @desrosj
6 years ago

  • Milestone changed from 5.0.3 to 5.1

This is not something that is broken, so let's keep it in 5.1.

This ticket was mentioned in Slack in #core-media by joemcgill. View the logs.


6 years ago

#8 @desrosj
6 years ago

  • Keywords needs-docs added

#9 @pento
6 years ago

  • Owner changed from swissspidy to pento

#10 @pento
6 years ago

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

In 44484:

Embeds: Polldaddy is now called Crowdsignal.

Crowdsignal adds an additional embed URL, survey.fm.

Props donncha.
Fixes #45036.

#11 @pento
6 years ago

For reference, the same rename is in progress in Gutenberg: https://github.com/WordPress/gutenberg/pull/12854

#12 @JustinSainton
6 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Line 175 of [44484] has a typo - s/Crowdsigna/Crowdsignal, I think?

#13 @swissspidy
6 years ago

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

In 44511:

Embeds: Fix typo introduced in [44484].

Props JustinSainton.
Fixes #45036.

Note: See TracTickets for help on using tickets.