Help Center: Logged-Out Support for Support Guides and Odie #107902
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
Contributor
|
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 |
Contributor
|
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: App Entrypoints (~502 bytes removed 📉 [gzipped]) DetailsCommon code that is always downloaded and parsed every time the app is loaded, no matter which route is used. Sections (~305 bytes removed 📉 [gzipped]) DetailsSections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to. Async-loaded Components (~289 bytes removed 📉 [gzipped]) DetailsReact 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. Generated by performance advisor bot at iscalypsofastyet.com. |
escapemanuele
force-pushed
the
logged-odie
branch
from
74f0bf9 to
6c4e2e2
Compare
Comment on lines
13
to
14
Contributor
There was a problem hiding this comment.
I'm also removing these as it looks like they are not used here
escapemanuele
force-pushed
the
logged-odie
branch
from
6c4e2e2 to
980d800
Compare
matticbot
added
the
[Status] Needs Review
heavyweight
reviewed
Jan 15, 2026
| persistValueSafely( 'logged_out_help_center_preferences', preferences ); | ||
| } else if ( isLoggedIn ) { | ||
| // Delete local preferences when logged in to avoid conflicts. | ||
| persistValueSafely( 'logged_out_help_center_preferences', null ); |
Contributor
There was a problem hiding this comment.
Can we put logged_out_help_center_preferences in a const somewhere? I see it being used a few times around
Comment on lines
314
to
330
| if ( ! supportInteraction && chatId && ! isLoggedOutSession ) { | ||
| supportInteraction = await startNewInteraction( { | ||
| event_external_id: chatId.toString(), | ||
| event_source: 'odie', | ||
| } ); | ||
| } else if ( supportInteraction && ! odieId && chatId ) { | ||
| } else if ( supportInteraction && ! odieId && chatId && ! isLoggedOutSession ) { | ||
| supportInteraction = await addEventToInteraction( { | ||
| interactionId: supportInteraction.uuid, | ||
| eventData: { | ||
| event_external_id: chatId.toString(), | ||
| event_source: 'odie', | ||
| }, | ||
| } ); | ||
| } else if ( isLoggedOutSession ) { | ||
| // If the user is not logged in, we don't need to create a new support interaction. | ||
| updateLoggedOutSession( chatId.toString(), returnedChat.session_id, botSlug ); | ||
| } |
Contributor
There was a problem hiding this comment.
What do you think of
if ( isLoggedOutSession ) {
// If the user is not logged in, we don't need to create a new support interaction.
updateLoggedOutSession( chatId.toString(), returnedChat.session_id, botSlug );
} else {
if ( ! supportInteraction && chatId ) {
...
}
My brain works better this way, totally up to you!
Member
Author
There was a problem hiding this comment.
Oh that's much better!
heavyweight
reviewed
Jan 16, 2026
Comment on lines
+17
to
+19
| window.localStorage.removeItem( PREFERENCES_KEY + 'help_center_open' ); | ||
| window.localStorage.removeItem( PREFERENCES_KEY + 'help_center_minimized' ); | ||
| window.localStorage.removeItem( PREFERENCES_KEY + 'help_center_router_history' ); |
Contributor
There was a problem hiding this comment.
NIT: slightly better for maintainability
Suggested change
| window.localStorage.removeItem( PREFERENCES_KEY + 'help_center_open' ); | |
| window.localStorage.removeItem( PREFERENCES_KEY + 'help_center_minimized' ); | |
| window.localStorage.removeItem( PREFERENCES_KEY + 'help_center_router_history' ); | |
| try { | |
| const keys = Object.keys( memoryStore ) as Array< keyof Preferences[ 'calypso_preferences' ] >; | |
| keys.forEach( ( key ) => { | |
| window.localStorage.removeItem( PREFERENCES_KEY + key ); | |
| } ); |
heavyweight
approved these changes
Jan 16, 2026
Contributor
heavyweight
left a comment
heavyweight
left a comment
There was a problem hiding this comment.
Didn't find any blockers
github-actions
bot
removed
the
[Status] Needs Review
heydemoura
pushed a commit
that referenced
this pull request
Jan 18, 2026
Co-authored-by: Emanuele Buccelli <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes dotcom-15390
Proposed Changes
This hooks up the Help Center and Odie client to the locally persisted chat and router states. In other words, this adds an alternative to Calypso preferences.
Why are these changes being made?
This allows logged out users to have coherent and continuous sessions.
Testing Instructions
Odie chats
Chat history
UI state
Chat continuity after logging in