-
Notifications
You must be signed in to change notification settings - Fork 808
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
Restore legacy publicize ui #13241
Restore legacy publicize ui #13241
Conversation
Caution: This PR has changes that must be merged to WordPress.com |
This is an automated check which relies on |
@gravityrail We are seeing the same tests failing in #13230, it seems something deep in core master. |
@gravityrail I'll give this a test once the checks pass and it shows up in the beta app! FYI, I'm AFK until wednesday though. |
@jeffgolenski tests fixed, let me know when you've taken a look. I'm interested to know if we can give this old UI a coat of paint so it fits in with modern Jetpack |
The functionality of the connections seems to work just fine (tested with linkedin). But we'll need to have a larger dicussion around overall UX / UI to bring this up to speed in 2019! |
This PR has been marked as stale. This happened because:
No further action is needed. But it's worth checking if this PR has clear testing instructions, is it up to date with master, and it is still valid. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation. |
WIP experiment.
cc @jeffgolenski