Paths

Table of Contentst

Differential D14036

[native][web] Show version unsupported modal when received use_new_flow on logout
ClosedPublic
Actions

Authored by bartek on Nov 25 2024, 11:52 AM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

kamil
varun

Commits

rCOMMa9ed6112cc26: [native][web] Show version unsupported modal when received use_new_flow on…

Summary

Address ENG-9718.
Display "version unsupported" message upon logout, when use_new_flow response is received

Depends on D14035

Test Plan

Mocked Identity logout response. I was visually logged out and modal appeared.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Nov 25 2024, 11:52 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 25 2024, 11:52 AM

bartek added a child revision: D14037: [native] Show version unsupported when opened QR screen.Nov 25 2024, 11:54 AM

Harbormaster completed remote builds in B32810: Diff 45997.Nov 25 2024, 12:41 PM

bartek published this revision for review.Nov 25 2024, 1:09 PM

kamil accepted this revision.Nov 26 2024, 10:13 AM

kamil added inline comments.

native/profile/profile-screen.react.js
579–581 ↗	(On Diff #45997)	I would memoize this object
web/settings/account-settings.react.js
48–53 ↗	(On Diff #45997)	I would use `React.useMemo` instead of `React.useCallback` and memorize entire `useLogOut` param

This revision is now accepted and ready to land.Nov 26 2024, 10:13 AM

bartek added inline comments.Nov 29 2024, 11:03 AM

web/settings/account-settings.react.js
48–53 ↗	(On Diff #45997)	Good idea, thank you

Apply hooks suggestion

Harbormaster completed remote builds in B32897: Diff 46128.Nov 29 2024, 11:30 AM

Closed by commit rCOMMa9ed6112cc26: [native][web] Show version unsupported modal when received use_new_flow on…. · Explain WhyNov 29 2024, 1:03 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMa9ed6112cc26: [native][web] Show version unsupported modal when received use_new_flow on….

ashoat added inline comments.Dec 2 2024, 6:10 PM

native/profile/profile-screen.react.js
579–581 ↗	(On Diff #45997)	This feedback doesn't seem to have been addressed

bartek mentioned this in D14078: [native] Memoize logOutOptions.Dec 4 2024, 3:44 PM

bartek mentioned this in rCOMM58cdeac3bb64: [native] Memoize logOutOptions.Dec 6 2024, 12:45 PM

Revision Contents
Changeset List

Path

Size

native/

profile/

profile-screen.react.js

7 lines

web/

settings/

account-settings.react.js

13 lines

Diff 46154

native/profile/profile-screen.react.js

Loading...

web/settings/account-settings.react.js

Loading...