Closed madhusudhand closed 1 day ago
https://calypso.live?image=registry.a8c.com/calypso/app:build-125153&env=jetpack |
https://calypso.live?image=registry.a8c.com/calypso/app:build-125153&env=a8c-for-agencies |
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 fix/rename-marketing-sharing-route
on your sandbox.
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:
Sections (~3 bytes added 📈 [gzipped])
Legend
Generated by performance advisor bot at iscalypsofastyet.com.
Going to merge and deploy this 👍
@madhusudhand can we add this to the CtD board? See p1732068120998669-slack-CRWCHQGUB.
Fixes: https://github.com/Automattic/dotcom-forge/issues/9868
Proposed Changes
Why are these changes being made?
/sites/marketing/sharing
conflicts with existing routes on dotcom, which returns a 302 redirect.Testing Instructions
/sites/marketing/sharing-buttons/:site
in browser. it should load the sharing content.Pre-merge Checklist