-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[EPIC] Jetpack and Nav Unification v1 prelaunch flow fixes #50225
Comments
Changing the checklist button link is handled in Automattic/jetpack#18843 |
This PR directly above should address the monitor notifications settings. For the items that mention Calypso Green vs Calypso Blue, those are handled via our redirects system, so can handle via a WPCOM commit. For the Calypso Add New Post task, I'm not sure that we can get to that. I'm not sure that we can get that handled before the launch, nor that we're the best placed to handle it. What's happening there, I believe, is that Calypso is trying to log the user in to their remote site via SSO so that we can then iFrame the editor in Calypso. This was built quite a while back when Atomic was launching. In the reported issue, I believe that SSO was likely not enabled. To fix this correctly, we would probably want to build something like magic login for Jetpack sites and then use that instead. But, realistically, I don't think that we could prioritize in the near term. Definitely not before the launch of nav unification in a few weeks. |
Yes I confirm the list above is the initial one I wrote with the fixes. |
Ajax will go through the Ajax items shortly and make sure we close any that have been completed. However, due to the Jetpack 9.5 code freeze having now past (on Monday) we'll have to make a call on what we can/can't fix via other means (ie: WPCOMSH) and then decide whether we can/can't include Jetpack connected sites in Nav Unification v1 launch. My gut instinct is that it's unlikely. I don't have confidence that JP support are happy it provides a good enough UX for all the edge cases yet. |
This patch D57273-code from @rcoll fixes the WP Admin Backup Credentials (along with Scan creds) and WP Admin Backup & Scan Nav items. |
@monsieur-z That looks positive. Would you be able to check off the items under #50225 that this addresses? Thank you. |
It will shortly: p1HpG7-blf-p2 |
I see D57273-code has been reverted in r221904-wpcom. I created separated issues for the remaining tasks (Automattic/jetpack#18975, Automattic/jetpack#18976), so we can close this epic out. |
The following are a set of "quick fixes" that will fix the flows for Jetpack connected sites for Nav Unification.
Given the other work that is happening simultaneously, there is no guarantee these will be fixable in time for launch. Therefore if they are unable to be completed, then a fallback will be to completely disable Nav Unification for Jetpack connected sites at the point of launch.
More context at p1HpG7-bcl-p2.
Preferred route: Fix the flows
To quote @folletto the aim of the changes is...
(see: p1HpG7-bcl-p2#comment-44627).
Guiding principles
Changes required
Our changes should be as follows. I would recommend these are broken up into their own Issues (update checkboxes with link) and close each one-by-one so we can have a record of where each point has been addressed:
From Calypso
Jetpack_Admin_Menu
so we don’t return more items that the ones currently present in the existing navigation. Admin Menu: Mimic existing Calypso menu on Jetpack sites jetpack#18921Import
,Export
,Feedback
, andWP Admin
links open in new tab and present an "external" icon. These are exceptions to the rule because they will be linking to WP Admin pages because there is no Calypso equivalent. Nav Unification: Open wp-admin links on new tabs (Jetpack sites only) #50444From WPAdmin
See p1HpG7-bcl-p2#comment-44716. Please note that Nav Unification has not caused any regressions here. This is simply an update that has become necessary as a result of the paradigm shift in the way we view the relationship between Calypso and WPAdmin introduced by Nav Unification.
Fallback: disable Nav Unification for launch
In the case that we cannot implement the above fixes in time for launch then we should fallback to...
Screenshots
This screenshot should be useful context
The text was updated successfully, but these errors were encountered: