Skip to content

API: Allow posts_per_rss and rss_use_excerpt to be updated/read from API#8513

Merged
zinigor merged 1 commit intomasterfrom
sync/jonathansadowski/r168133-wpcom
Jan 29, 2018
Merged

API: Allow posts_per_rss and rss_use_excerpt to be updated/read from API#8513
zinigor merged 1 commit intomasterfrom
sync/jonathansadowski/r168133-wpcom

Conversation

@jonathansadowski
Copy link
Copy Markdown
Contributor

This Diff updates the site settings endpoints to allow posts_per_rss and rss_use_excerpt to be updated and read from the API.

See p3Ex-32f-p2.

Differential Revision: D9173-code

This commit syncs r168133-wpcom.

Testing instructions:

  • Apply this PR to a Jetpack site managed in Calypso
  • In Calypso, visit Site-Settings for this site and select "Writing" (https://wordpress.com/settings/writing/)
  • A section for "Feed Settings" should appear (it wouldn't have appeared prior to applying this PR).
  • Any changes made to feed settings should persist to your site.

@mattsherman
Copy link
Copy Markdown

This PR has been outstanding for two weeks. Is there anything we need to do, or is this length of time to be expected? Just want to make sure this doesn't fall through the cracks!

@jeherve jeherve added this to the 5.8 milestone Jan 29, 2018
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.

LGTM! 🚢

@jeherve jeherve 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 Jan 29, 2018
@zinigor zinigor merged commit f4c1906 into master Jan 29, 2018
@zinigor zinigor deleted the sync/jonathansadowski/r168133-wpcom branch January 29, 2018 10:37
@zinigor zinigor removed the [Status] Ready to Merge Go ahead, you can push that green button! label Jan 29, 2018
jeherve added a commit that referenced this pull request Jan 29, 2018
zinigor pushed a commit that referenced this pull request Jan 30, 2018
* Changelog 5.8: create base for changelog.

* Update 5.8 release post link

* fix 5.8 release date

* Updates to plugin description

* Changelog: add #8499

* Changelog: add #8506

* Changelog: add #8509

* Changelog: add #8516

* Changelog: add #8517

* Changelog: add #8523

* Changelog: add #8547

* Changelog: add #8496

* Changelog: add #8584

* Changelog: add #8595

* Changelog: add #8445

* Changelog: add #8431

* Changelog: add #8284

* Changelog: add #8270

* Changelog: add #8124

* Changelog: add #8581

* Changelog: add #8463

* Changelog: add #8568 (#8646)

* Updates to testing list and changelog

* Changelog: add #8443

* Changelog: add #8459

* Changelog: add #8469

* Changelog: add #8464

* Changelog: add #8478 and #8479

* Changelog: add #8483

* Changelog: add #8488

* Changelog: add #8513

* Changelog: add #8555

* Changelog: add #8565

* Changelog: add #8601

* Changelog: add #8612

* Changelog: add first pass at Search items.

* Changelog: add more info to help test Search.

* Changelog: add #8144

* Changelog: add #8313

* Changelog: add #8419

* Changelog: add #8465

* Changelog: add #8515

* Changelog: add #8587

* Changelog: add #8591

* Changelog: add #8659

* Changelog: add #8661

* Changelog: add #8671

* Changelog: add 5.7.1 to archived changelog too.

* Reverted changes to readme, removed entry about backups.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants