Differential D11844

[web] display modal explaining why user was logged out
ClosedPublic
Actions

Authored by varun on Apr 30 2024, 9:39 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Nov 11, 5:45 PM

	Unknown Object (File)
	Mon, Nov 11, 12:31 PM

	Unknown Object (File)
	Mon, Nov 11, 9:59 AM

	Unknown Object (File)
	Mon, Nov 11, 12:53 AM

	Unknown Object (File)
	Oct 15 2024, 9:05 PM

	Unknown Object (File)
	Oct 10 2024, 9:00 PM

	Unknown Object (File)
	Oct 10 2024, 9:00 PM

	Unknown Object (File)
	Oct 10 2024, 9:00 PM

View All 48 Files

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMM4dcf358d2c31: [web] display modal explaining why user was logged out

Summary

when web clients receive the first code version with usingCommServicesAccessToken set to true, we will need to log them out so that they can register/log in to the identity service and get a CSAT. when we log them out, we should display a modal explaining what has happened.

Depends on D11705

Test Plan

set the usingCSAT flag to true and refreshed the app. confirmed that modal was displayed and the link worked

Screenshot 2024-04-30 at 12.51.19 PM.png (1×1 px, 154 KB)

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Apr 30 2024, 9:39 AM

varun edited the test plan for this revision. (Show Details)Apr 30 2024, 9:52 AM

varun attached a referenced file: F1683797: Screenshot 2024-04-30 at 12.51.19 PM.png. (Show Details)

Harbormaster completed remote builds in B28605: Diff 39707.Apr 30 2024, 9:56 AM

varun requested review of this revision.Apr 30 2024, 9:56 AM

ashoat accepted this revision.Apr 30 2024, 10:05 AM

This revision is now accepted and ready to land.Apr 30 2024, 10:05 AM

varun added a child revision: D11860: [web] differentiate between password and wallet users in MissingCSATModal.May 2 2024, 9:29 PM

Closed by commit rCOMM4dcf358d2c31: [web] display modal explaining why user was logged out (authored by varun). · Explain WhyMay 5 2024, 5:29 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM4dcf358d2c31: [web] display modal explaining why user was logged out.

Revision Contents
Changeset List

Path

Size

web/

2 lines

components/

log-out-if-missing-csat-handler.react.js

43 lines

2 lines

Diff 39707

web/app.react.js

Loading...

web/components/log-out-if-missing-csat-handler.react.js

Loading...

web/root.js

Loading...