Skip to content

Settings UI: Adds sharing tab with cards for individual sharing services.#6750

Merged
dereksmart merged 7 commits intofeature/settings-overhaulfrom
add/sharing-tab
Mar 24, 2017
Merged

Settings UI: Adds sharing tab with cards for individual sharing services.#6750
dereksmart merged 7 commits intofeature/settings-overhaulfrom
add/sharing-tab

Conversation

@dereksmart
Copy link
Copy Markdown
Contributor

Fixes: #6745

This adds a new route/tab for /sharing

screen shot 2017-03-24 at 4 02 52 pm

Users who are linked should be taken to wordpress.com/sharing settings area, where unlinked users should see cards asking them to connect their user account.

It has three sub-modules: publicize, sharing, and likes.  Likes is the only one with actual settings. The others will link to wordpress.com to either configure settings there, or link the user account if not linked.
@dereksmart dereksmart added Admin Page React-powered dashboard under the Jetpack menu [Feature] Likes [Feature] Publicize Now Jetpack Social, auto-sharing [Feature] Sharing Post sharing, sharing buttons [Status] Needs Design Review Design has been added. Needs a review! [Status] Needs Review This PR is ready for review. labels Mar 24, 2017
@dereksmart dereksmart added this to the Settings UI milestone Mar 24, 2017
@rickybanister
Copy link
Copy Markdown

Can the tab be between writing and discussion?

Otherwise looks awesome.

@dereksmart
Copy link
Copy Markdown
Contributor Author

Moved the tab, and also fixed so that editors will have access to the publicize card.

@eliorivero
Copy link
Copy Markdown
Contributor

eliorivero commented Mar 24, 2017

Got errors on yarn build and it doesn't build

/wp-content/plugins/jetpack/_inc/client/sharing/publicize.jsx
  49:2  error  Newline required at end of file but not found  eol-last

✖ 1 problem (1 error, 0 warnings)

Details:
    module: Module[1109]
    error: 
/wp-content/plugins/jetpack/_inc/client/sharing/publicize.jsx
  49:2  error  Newline required at end of file but not found  eol-last

✖ 1 problem (1 error, 0 warnings)

</SettingsCard>
);
}
} No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Needs moar line breaks

Copy link
Copy Markdown
Contributor Author

@dereksmart dereksmart Mar 24, 2017

Choose a reason for hiding this comment

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

done b93574c and fixed others

@eliorivero
Copy link
Copy Markdown
Contributor

Good to go!

@dereksmart dereksmart merged commit 1e1e62b into feature/settings-overhaul Mar 24, 2017
@dereksmart dereksmart deleted the add/sharing-tab branch March 24, 2017 21:59
@dereksmart dereksmart removed the [Status] Needs Review This PR is ready for review. label Mar 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Admin Page React-powered dashboard under the Jetpack menu [Feature] Likes [Feature] Publicize Now Jetpack Social, auto-sharing [Feature] Sharing Post sharing, sharing buttons [Status] Needs Design Review Design has been added. Needs a review!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants