Automattic / wp-calypso

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

Strip trailing slash from source URL in credentials form #96649

Open donnapep opened 7 hours ago

donnapep commented 7 hours ago

Fixes #96416.

Proposed Changes

Strips trailing slash, if applicable, from Current site address field on DIFM credentials form.

Screenshot 2024-11-21 at 3 27 09 PM

Testing Instructions

Pre-merge Checklist

github-actions[bot] commented 7 hours ago
Calypso Live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125324
Jetpack Cloud live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125324&env=jetpack
Automattic for Agencies live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125324&env=a8c-for-agencies
matticbot commented 7 hours 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 fix/strip-trailing-slash-from-source-url on your sandbox.

matticbot commented 6 hours ago

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.