Skip to content

Use platform.js for Google+ sharing functionality#8129

Merged
jeherve merged 1 commit intoAutomattic:masterfrom
marksamman:master
Nov 21, 2017
Merged

Use platform.js for Google+ sharing functionality#8129
jeherve merged 1 commit intoAutomattic:masterfrom
marksamman:master

Conversation

@marksamman
Copy link
Copy Markdown
Contributor

Changes proposed in this Pull Request:

plusone.min.js and platform.min.js are currently broken, gapi.plusone is undefined. The documentation at https://developers.google.com/+/web/+1button/ uses platform.js, which is working fine.

@marksamman marksamman requested a review from a team as a code owner November 8, 2017 14:04
@jeherve jeherve added [Feature] Sharing Post sharing, sharing buttons [Pri] High [Status] Needs Review This PR is ready for review. Bug When a feature is broken and / or not performing as intended labels Nov 8, 2017
@jeherve
Copy link
Copy Markdown
Member

jeherve commented Nov 10, 2017

Also reported in 780657-zen

@richardmtl
Copy link
Copy Markdown
Contributor

Just popping in to say that the folks in this thread say that the patch works: https://wordpress.org/support/topic/5-5-official-google-button-not-showing/

@zinigor
Copy link
Copy Markdown
Contributor

zinigor commented Nov 20, 2017

Reverts #7912

Copy link
Copy Markdown
Contributor

@zinigor zinigor left a comment

Choose a reason for hiding this comment

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

Looks good to me, thank you!

@zinigor zinigor added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Nov 20, 2017
@zinigor zinigor added this to the 5.5.1 milestone Nov 20, 2017
Copy link
Copy Markdown
Member

@jeherve jeherve left a comment

Choose a reason for hiding this comment

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

Seems to work on my end. Merging!

@jeherve jeherve merged commit 061e27f into Automattic:master Nov 21, 2017
@jeherve jeherve removed the [Status] Ready to Merge Go ahead, you can push that green button! label Nov 21, 2017
@jeherve
Copy link
Copy Markdown
Member

jeherve commented Nov 21, 2017

Cherry-picked to branch-5.5 in 85c495b

jeherve added a commit that referenced this pull request Nov 21, 2017
jeherve pushed a commit that referenced this pull request Nov 22, 2017
* Added changelog for #8201 and #8177.

* Changelog 5.5.1: create base for changelog.

* Fix typo and update release post link.

* Changelog: add #8167

* Changelog: add #8204

* Changelog: add #8129

* Changelog: add #8225

* Changelog: add #8219
jeherve pushed a commit that referenced this pull request Nov 22, 2017
* Added changelog for #8201 and #8177.

* Changelog 5.5.1: create base for changelog.

* Fix typo and update release post link.

* Changelog: add #8167

* Changelog: add #8204

* Changelog: add #8129

* Changelog: add #8225

* Changelog: add #8219
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug When a feature is broken and / or not performing as intended [Feature] Google+ [Feature] Sharing Post sharing, sharing buttons [Pri] BLOCKER Touches WP.com Files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants