Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change PollDaddy embed to Crowdsignal #13180

Closed
wants to merge 4 commits into from

Conversation

geekpulp
Copy link
Contributor

@geekpulp geekpulp commented Jan 3, 2019

Description

Change the Polldaddy embed block to Crowdsignal (Their new brand) by updating core-embeds.js.

Fixes #11517.

How has this been tested?

Tested in Safari 12.0.2, added a poll to the page using the embed. updated branded presented correctly

The standard docker environment was used for testing

This change doesn't affect other areas of the code

Screenshots

Types of changes

Content change for updated branding of a service

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.

Copy link
Member

@Soean Soean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your PR. This code is not working for existing polldaddy blocks. For testing this code please create a post with the old polldaddy block. After the code change, the block should be auto-transformed into the new block.

packages/block-library/src/embed/core-embeds.js Outdated Show resolved Hide resolved
@Soean Soean added the [Block] Embed Affects the Embed Block label Jan 3, 2019
@geekpulp geekpulp changed the title Change polldaddy embed to Crowdsignal Change PollDaddy embed to Crowdsignal Jan 4, 2019
@@ -0,0 +1,2 @@
{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this file should be added?!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed. @geekpulp You should add .vscode to your global .gitignore file :-)

@aduth
Copy link
Member

aduth commented Jan 4, 2019

Duplicate of #12854 ?

@swissspidy swissspidy added the [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed label Jan 4, 2019
@swissspidy
Copy link
Member

Indeed a duplicate.

@swissspidy swissspidy closed this Jan 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Embed Affects the Embed Block [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants