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
My Home: Add Store education card #61586
Conversation
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: Sections (~179 bytes added 📈 [gzipped])
Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to. 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. |
Behaviorally this looks great - however, the requirements for the issue state we need to display this specifically for users with the "sell" intent. It doesn't look like we're checking for that right now; is that intended for a future PR? |
Aha, I missed that! I will fix that, as well as updating the illustration to use the correct one :) |
@blackjackkent I updated the server-side diff to add a check for the intent before displaying the card! |
links={ [ | ||
{ | ||
externalLink: true, | ||
url: `https://wordpress.com/support/video-tutorials-add-payments-features-to-your-site-with-our-guides/`, |
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.
This may need localizeUrl
, need to double check.
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.
It looks like this content hasn't been translated to other languages yet.
added a comment on D75890-code |
4e82145
to
69a3a30
Compare
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.
This Pull Request is now available for translation here: https://translate.wordpress.com/deliverables/7171556 Thank you @sixhours for including a screenshot in the description! This is really helpful for our translators. |
Translation for this Pull Request has now been finished. |
Changes proposed in this Pull Request
sell
intentVisual demo
Testing instructions
/home
on a launched site that's created through the seller experience flowpublic-api.wordpress.com
Related to #61467