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

Site Settings: Separate Composing in a separate component #11192

Merged
merged 1 commit into from
Feb 7, 2017

Conversation

tyxla
Copy link
Member

@tyxla tyxla commented Feb 6, 2017

This PR moves the Composing card to a separate component. This will allow us to add more settings without driving the writing form component grow too large. There are no functional changes, this PR only prepares the card for adding more settings that are specific to Jetpack sites.

This PR is part of #9171.

To test:

  • Checkout this branch.
  • Go to /settings/writing/$site for a Jetpack site.
  • Verify all settings in the Composing are retrieved and saved correctly, and there are no regressions in that card.
  • Go to /settings/writing/$site for a WordPress.com site.
  • Verify all settings in the Composing are retrieved and saved correctly, and there are no regressions in that card.

@tyxla tyxla added [Feature] Site Settings All other general site settings. [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Feb 6, 2017
@tyxla tyxla added this to the Jetpack Module Settings in Calypso milestone Feb 6, 2017
@tyxla tyxla self-assigned this Feb 6, 2017
@matticbot
Copy link
Contributor

@oskosk oskosk mentioned this pull request Feb 6, 2017
55 tasks
Copy link
Member

@ryelle ryelle 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 👍

@roccotripaldi roccotripaldi added [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Feb 6, 2017
@tyxla tyxla force-pushed the update/site-settings-composing-separate-component branch from 37b42c6 to e49717f Compare February 7, 2017 08:23
@tyxla tyxla merged commit 14f3654 into master Feb 7, 2017
@tyxla tyxla deleted the update/site-settings-composing-separate-component branch February 7, 2017 08:28
@oskosk oskosk mentioned this pull request Mar 1, 2017
48 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Site Settings All other general site settings.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants