Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.43k stars 1.99k forks source link

Stats: Change support links to open in a new tab #96574

Open jwebbdev opened 2 days ago

jwebbdev commented 2 days ago

Related to https://github.com/Automattic/red-team/issues/199

Proposed Changes

Why are these changes being made?

Testing Instructions

Screenshot 2024-11-19 at 11 02 05 PM

Pre-merge Checklist

github-actions[bot] commented 2 days ago
Calypso Live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125296
Jetpack Cloud live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125296&env=jetpack
Automattic for Agencies live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125296&env=a8c-for-agencies
matticbot commented 2 days ago

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 install-plugin.sh $pluginSlug update/stats-external-support-links on your sandbox.

matticbot commented 2 days ago

Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:

Sections (~8 bytes added 📈 [gzipped])

``` name parsed_size gzip_size stats +1122 B (+0.1%) +8 B (+0.0%) ``` Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to.

Async-loaded Components (~36 bytes added 📈 [gzipped])

``` name parsed_size gzip_size async-load-calypso-my-sites-stats-pages-insights +132 B (+0.2%) +30 B (+0.2%) async-load-calypso-my-sites-stats-pages-subscribers +33 B (+0.0%) +6 B (+0.0%) ``` React components that are loaded lazily, when a certain part of UI is displayed for the first time.

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. **Gzip Size:** Compressed size of the JS and CSS files. This much data needs to be downloaded over network.

Generated by performance advisor bot at iscalypsofastyet.com.

kangzj commented 1 day ago

It looks good to me. I just wonder why the builds are failing for e2e tests🤔 I restarted them multiple times but no use.

jwebbdev commented 17 hours ago

@kangzj I hadn't run the install-plugin.sh $pluginSlug update/stats-external-support-links items referenced. Not sure if that was the issue (though it seems it just pulls that from TC, not push... so still unclear) as after doing that and rerunning the failed TeamCity jobs they still failed. So I merged trunk and pushed to trigger another full workflow and it seemed to work.

kangzj commented 48 minutes ago

nah you don’t need to run that command. The command is for Odyssey Stats testing.

Thanks for fixing these! Very much appreciated! 🙇