-
Notifications
You must be signed in to change notification settings - Fork 2k
[NOX In-Context] Do not show the Jetpack logo when onboarding for Woo accounts #103213
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
Conversation
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
|
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: App Entrypoints (~4 bytes added 📈 [gzipped]) DetailsCommon code that is always downloaded and parsed every time the app is loaded, no matter which route is used. Legend What is parsed and gzip size?Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. Generated by performance advisor bot at iscalypsofastyet.com. |
dmallory42
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this! I can see it looks as expected with the typo fixed and the logo showing correctly. Let's ship it! 🚢
|
This PR modifies the release build for the following Calypso Apps: For info about this notification, see here: PCYsg-OT6-p2
To test WordPress.com changes, run |
|
Translation for this Pull Request has now been finished. |
|
Hi! I had to fix some merge conflicts in #102811 due to this PR, and wanted to check if these changes weren't affected, but I'm not sure how to test this. Assume I know nothing about Woo 😅 How do I create a JN site from WooCommerce core? And where can I find a zip of WooPayments 9.3.0? |
|
Hi @tellthemachines! To create a JN site from WooCommerce Core and WooPayments 9.3, you can go to Jurassic Ninja, log in to your WP account, and then click Let us know if we can help further! |
|
Thanks @dmallory42! I've tested and all looks good. |
* Update content area of login screens to use StepContainerV2 * Dead code removal * Untangle Gravatar and WP Job Manager logins from isWhiteLogin * Extract header text rendering logic from LoginBlock * Correct header text logic and wp cloud logo * Move things around and re-style them * Use component alignment * Remove extra padding and update tos * Additional merge-related changes * Remove continue as user height * Remove convoluted button widths * Fix scrollbar * Improve syntax * Remove positioning from "Not you" * Update `showContinueAsUser` to optional boolean in getHeaderText * Fix use of `is-white-login` in magic login component * Fix height in magic login * Update header condition to isWhiteLogin * Fix weird a4a login screen * Remove privateSite mentions * Reapply changes from #103213 * Fix typo * Fix margin for 2fa screen
Fixes woocommerce/woocommerce#57799
Proposed Changes
This fixes two main problems:
This PR fixes both issues.
Why are these changes being made?
To fix the logo issue (and typo as a bonus)
Testing Instructions
http://calypso.localhost:3000/https://wordpress.com/withhttp://calypso.localhost:3000/. You should see the same login page but with one logo only (and a fixed typo):Pre-merge Checklist