-
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
Advertising A4A: Add touchpoint to sites dashboard #90963
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 (~51 bytes removed 📉 [gzipped])
Common code that is always downloaded and parsed every time the app is loaded, no matter which route is used. Sections (~4533 bytes removed 📉 [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. |
cef068a
to
57b2cef
Compare
Hi @taipeicoder. We talked about adding a banner to this dashboard a little while ago p1715217336449049/1715217104.318879-slack-C06DN6QQVAQ, and you had mentioned a potentially broken layout if we did so. From what I can tell things look good, but lemme know if there are any issues :D Dave Martin had also mentioned a command + k promo at the top of the screen, but I'm having a bit of a hard time finding it in the sites-dashboard-v2 codebase. Wondering if you'd mind pointing me in the right direction 🙏 |
LGTM!
We need to make sure that we don't place two notices at the same time. |
Thanks @lucasmendes-design!
It sounds like it may have been removed, but I'll spend a bit more time searching for it just in case p1716323212763989/1715217104.318879-slack-C06DN6QQVAQ |
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.
The problem here is that we have the maximum width of the table. I believe this is the problem. Ccing @davemart-in for thoughts. I'm not sure who developed this so we can have the same width for both. |
@lucasmendes-design this is related to this message I shared in Slack this morning: p1716888347135899/1716814114.968029-slack-C06DN6QQVAQ Once we have consensus on that (one way or the other) we can move this forward. |
Update: We are keeping the max-width on the tables.
@jeyip awesome. Thanks! |
12e1be7
to
faf1d90
Compare
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 |
Related to https://github.com/Automattic/martech/issues/3002
Proposed Changes
Why are these changes being made?
As part of the release of the Automattic for Agencies initiative ( A4A ), the A4A team is implementing the ability for users to purchase plans in bulk for a discount pfunGA-1a8-p2. A4A, however, is a relatively new platform, meaning that folks will be otherwise unaware of its existence, let alone the bulk plans purchasing feature A4A offers.
Because of this, we’d like to promote the bulk purchasing feature and drive traffic to the site through different logged-out and logged-in touchpoints on WordPress.com.
Testing Instructions
/sites
as a user with 5 or more sitesPre-merge Checklist