Skip to content

Sharing: save post meta on the media edit screen as well.#10152

Merged
jeherve merged 1 commit intomasterfrom
fix/sharing-attachments-10150
Sep 24, 2018
Merged

Sharing: save post meta on the media edit screen as well.#10152
jeherve merged 1 commit intomasterfrom
fix/sharing-attachments-10150

Conversation

@jeherve
Copy link
Copy Markdown
Member

@jeherve jeherve commented Sep 17, 2018

Fixes #10150

Changes proposed in this Pull Request:

Until now, we only saved post meta when saving a post, not when saving an attachment.
This problem does not exist with Likes as we already hook into edit_attachment there.

Testing instructions:

  1. Go to WP-Admin > Media > Open an image
  2. Hit edit more details option
  3. Disable sharing buttons from the right sidebar
  4. Save settings
  5. Make sure your settings are saved properly.

Proposed changelog entry for your changes:

Sharing: allow saving sharing button options on media edit page as well.

Fixes #10150

Until now, we only saved post meta when saving a post, not when saving an attachment.
This problem does not exist with Likes as we already hook into edit_attachment there.
@jeherve jeherve added Bug When a feature is broken and / or not performing as intended [Feature] Sharing Post sharing, sharing buttons [Status] Needs Review This PR is ready for review. labels Sep 17, 2018
@jeherve jeherve self-assigned this Sep 17, 2018
@jeherve jeherve requested a review from a team as a code owner September 17, 2018 07:26
@jeherve jeherve added this to the 6.6 milestone Sep 17, 2018
@jetpackbot
Copy link
Copy Markdown
Collaborator

That's a great PR description, thank you so much for your effort!

Generated by 🚫 dangerJS

Copy link
Copy Markdown
Contributor

@kraftbj kraftbj left a comment

Choose a reason for hiding this comment

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

:shipit:

@kraftbj kraftbj 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 Sep 24, 2018
@jeherve jeherve merged commit 8dc1c88 into master Sep 24, 2018
@jeherve jeherve deleted the fix/sharing-attachments-10150 branch September 24, 2018 15:27
@ghost ghost removed the [Status] Ready to Merge Go ahead, you can push that green button! label Sep 24, 2018
jeherve added a commit that referenced this pull request Sep 24, 2018
jeherve added a commit that referenced this pull request Sep 25, 2018
* Readme: add boilerplate for next release, 6.6

* Add 6.5 to the changelog.txt file

* Set boilerplate testing list for 6.6

* Readme: update stable tag to 6.5

* Add bullets to 6.5 changelog items

* Readme: add link to previous changelogs

This will help folks who want to know more about past releases,
while keeping the readme.txt short so as to not overwhelm translators and site owners only looking for information about the last release.

* Changelog: add information at the top of the changelog file.

* Changelog: add #10054

* Changelog: add #10078

* Changelog: add #10079

* Changelog: add #10064

* Changelog: add #10094

* Changelog: add #10096

* Testing list: add more information based on #10087

* Changelog: add #9847

* Changelog: add #10084

* Changelog: add #9918

* Changelog: add #7614

* Changelog: add #10116

* Changelog: add #10108

* Changelog: add #10041

* Changelog: add #10121

* Changelog: add #10134

* Changelog: add #10130

* Changelog: add #10109

* changelog: add #10137

* changelog: add #9952

* changelog: add #10120

* changelog: add #10162

* Changelog: add #10163

* Changelog: add #10092

* changelog: add #10156

* Changelog: add #10154

* changelog: add #10122

* Changelog: add #10101

* changelog: add #10105

* changelog: add #10190

* Changelog: add #10196

* changelog: add #10152

* Changelog: add #10153

* Testing list: add more details to Site Verification testing steps.

@see #10143 (comment)

* changelog: add #10194

* Changelog: add #10193
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] Sharing Post sharing, sharing buttons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants