Page MenuHomePhabricator
Differential D12406

[lib][native] add useVersionSupportedByIdentity hook and use it on native
ClosedPublic
Actions

Authored by varun on Jun 12 2024, 8:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 14, 8:48 AM
Unknown Object (File)
Mon, Nov 11, 7:48 AM
Unknown Object (File)
Mon, Nov 11, 7:28 AM
Unknown Object (File)
Sun, Nov 10, 6:25 AM
Unknown Object (File)
Sun, Nov 10, 1:43 AM
Unknown Object (File)
Wed, Nov 6, 2:59 PM
Unknown Object (File)
Wed, Nov 6, 1:54 AM
Unknown Object (File)
Mon, Nov 4, 1:37 AM
View All Files
Subscribers
tomek

Details

Reviewers
ashoat
Commits
rCOMM62c362d0a861: [lib][native] add useVersionSupportedByIdentity hook and use it on native
Summary

we should avoid calling commRustModule directly from VersionSupportedChecker

Depends on 12405

Test Plan

ran the iOS simulator and confirmed that the VERSION_SUPPORTED_BY_IDENTITY_STARTED and VERSION_SUPPORTED_BY_IDENTITY_SUCCESS actions were dispatched.

modified staging identity to set the min supported version to 1000, confirmed that the alert was displayed on simulator

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun created this revision.Jun 12 2024, 8:22 AM
Herald added a subscriber: tomek. · View Herald TranscriptJun 12 2024, 8:22 AM
Harbormaster completed remote builds in B29625: Diff 41244.Jun 12 2024, 8:46 AM
varun requested review of this revision.Jun 12 2024, 8:46 AM
ashoat accepted this revision.Jun 12 2024, 9:49 AM
ashoat added inline comments.
lib/types/redux-types.js
1500 ↗(On Diff #41244)

Making the payload an object makes it easier to modify later

This revision is now accepted and ready to land.Jun 12 2024, 9:49 AM
ashoat added a parent revision: D12405: [native][web][lib] add versionSupported method to IdentityServiceClient.Jun 12 2024, 9:49 AM
varun added a child revision: D12407: [web] check if version is supported by identity service.Jun 12 2024, 12:20 PM
varun updated this revision to Diff 41337.Jun 14 2024, 8:00 AM

make success payload an object

Harbormaster failed remote builds in B29684: Diff 41337!Jun 14 2024, 8:23 AM
Harbormaster completed remote builds in B29684: Diff 41337.Jun 14 2024, 8:50 AM
Closed by commit rCOMM62c362d0a861: [lib][native] add useVersionSupportedByIdentity hook and use it on native (authored by varun). · Explain WhyJun 17 2024, 9:47 AM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMM62c362d0a861: [lib][native] add useVersionSupportedByIdentity hook and use it on native.

Revision Contents
Changeset List

PathSize
lib/
actions/
20 lines
types/
18 lines
native/
components/
22 lines

Diff 41337

View Options

lib/actions/user-actions.js

Loading...
View Options

lib/types/redux-types.js

Loading...
View Options

native/components/version-supported.react.js

Loading...