Page MenuHomePhabricator
Differential D8874

[native] Memorize ChatContextProvider
ClosedPublic
Actions

Authored by • patryk on Aug 19 2023, 10:22 AM.
Tags
None
Referenced Files
F5933187: D8874.id.diff
Fri, Apr 18, 5:32 AM
F5894578: D8874.id30105.diff
Thu, Apr 17, 10:29 AM
Unknown Object (File)
Fri, Apr 11, 9:04 PM
Unknown Object (File)
Wed, Apr 9, 1:03 AM
Unknown Object (File)
Tue, Apr 8, 11:19 PM
Unknown Object (File)
Tue, Apr 8, 9:21 PM
Unknown Object (File)
Thu, Mar 27, 9:04 AM
Unknown Object (File)
Sun, Mar 23, 10:17 AM
View All 99 Files
Subscribers
ashoat
tomek

Details

Reviewers
tomek
inka
Commits
rCOMMb2d2f732a29b: [native] Memorize ChatContextProvider
Summary

D8847 changes ChatContextProvider position in root.react.js. We can also wrap this provider in React.memo.

Depends on D8847.

Test Plan

Run native app and check if everything works properly.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• patryk created this revision.Aug 19 2023, 10:22 AM
• patryk held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 19 2023, 10:22 AM
Harbormaster completed remote builds in B21918: Diff 30105.Aug 19 2023, 10:40 AM
• patryk edited the summary of this revision. (Show Details)Aug 19 2023, 11:03 AM
• patryk edited the test plan for this revision. (Show Details)
• patryk added reviewers: tomek, inka.
• patryk edited the summary of this revision. (Show Details)Aug 19 2023, 3:29 PM
• patryk added a parent revision: D8847: [native] Change ChatContextProvider position.
• patryk added a child revision: D8849: [native] Add chat mention rule to text message rules.
• patryk mentioned this in D8847: [native] Change ChatContextProvider position.Aug 20 2023, 7:56 AM
• patryk edited the summary of this revision. (Show Details)Aug 20 2023, 1:54 PM
• patryk published this revision for review.Aug 20 2023, 11:13 PM
tomek accepted this revision.Sep 12 2023, 1:56 AM
This revision is now accepted and ready to land.Sep 12 2023, 1:56 AM
Closed by commit rCOMMb2d2f732a29b: [native] Memorize ChatContextProvider (authored by • patryk). · Explain WhySep 28 2023, 1:40 AM
This revision was automatically updated to reflect the committed changes.
• patryk added a commit: rCOMMb2d2f732a29b: [native] Memorize ChatContextProvider.

Revision Contents
Changeset List

PathSize
native/
chat/
5 lines

Diff 31491

View Options

native/chat/chat-context-provider.react.js

Show First 20 Lines • Show All 79 Lines • ▼ Show 20 Lines "dependencies": {
"redux-devtools-extension": "^2.13.2", "redux-devtools-extension": "^2.13.2",
"redux-persist": "^6.0.0", "redux-persist": "^6.0.0",
"redux-thunk": "^2.2.0", "redux-thunk": "^2.2.0",
"reselect": "^4.0.0", "reselect": "^4.0.0",
"simple-markdown": "^0.7.2", "simple-markdown": "^0.7.2",
"siwe": "^1.1.6", "siwe": "^1.1.6",
"sql.js": "^1.8.0", "sql.js": "^1.8.0",
"tinycolor2": "^1.4.1", "tinycolor2": "^1.4.1",
"uuid": "^3.3.3", "uuid": "^3.4.0",
"visibilityjs": "^2.0.2", "visibilityjs": "^2.0.2",
"wagmi": "^0.8.10" "wagmi": "^0.8.10"
}, },
"jest": { "jest": {
"roots": [ "roots": [
"<rootDir>" "<rootDir>"
], ],
"transform": { "transform": {
Show All 13 Lines