[lib] Let useInvalidCSATLogOut run once
ClosedPublic
Actions

Authored by bartek on Jul 11 2025, 7:25 AM.

Details

Reviewers

kamil
tomek

Commits

rCOMM338ecb167ade: [lib] Let useInvalidCSATLogOut run once

Summary

Now with additional alerting introduced in D14879, I see that when force-logged-out on invalid CSAT, I'm receiving multiple alerts.
This hook needs to be called once at a time, so introduced a promiseRef pattern.

Test Plan

Confirmed that now only one alert is displayed when I'm logged out.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Jul 11 2025, 7:25 AM

bartek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJul 11 2025, 7:25 AM

Harbormaster completed remote builds in B34529: Diff 48688.Jul 11 2025, 7:44 AM

bartek published this revision for review.Jul 11 2025, 8:45 AM

kamil accepted this revision.Jul 14 2025, 11:52 AM

This revision is now accepted and ready to land.Jul 14 2025, 11:52 AM

Closed by commit rCOMM338ecb167ade: [lib] Let useInvalidCSATLogOut run once. · Explain WhyJul 16 2025, 9:53 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM338ecb167ade: [lib] Let useInvalidCSATLogOut run once.

Revision Contents
Changeset List

Path

Size

lib/

actions/

user-actions.js

37 lines

Diff 48688

View Options

lib/actions/user-actions.js

[lib] Let useInvalidCSATLogOut run onceClosedPublicActions