Closed epeicher closed 4 days ago
https://calypso.live?image=registry.a8c.com/calypso/app:build-124939&env=jetpack |
https://calypso.live?image=registry.a8c.com/calypso/app:build-124939&env=a8c-for-agencies |
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:
Sections (~49 bytes added 📈 [gzipped])
Legend
Generated by performance advisor bot at iscalypsofastyet.com.
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/perf-profiler-llm-click-track-event
on your sandbox.
Resolves https://github.com/Automattic/dotcom-forge/issues/9873
Proposed Changes
url
,hash
andchatId
tocalypso_performance_profiler_llm_survey_click
track eventPlease note: I didn't add the
message_id
to the track event because with thechat_id
, the messages are very easy to identify (currently there is only one message), and I don't think it is necessary. But if required, it would be straightforward to add it.Why are these changes being made?
Testing Instructions
/speed-test-tool?hash=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpZCI6NzA3MX0.m3-63iGZmpJOpOfkZ4ww184EoKd8P3UHzTYwv1VLQUo&url=https%3A%2F%2Fwordpress.com%2F&tab=mobile
t.gif
, additionally, use the Tracks Vigilante or any other tool to inspect the Tracks events.good
orbad
calypso_performance_profiler_llm_survey_click
track event is sent and it contains the properurl
,hash
andchatId
Pre-merge Checklist