Page MenuHomePhabricator

[native] Fetch invite links when entering thread settings
ClosedPublic

Authored by inka on Jun 3 2024, 8:33 AM.
Tags
None
Referenced Files
F3495776: D12286.id40928.diff
Thu, Dec 19, 9:58 AM
F3495775: D12286.id40968.diff
Thu, Dec 19, 9:58 AM
F3495774: D12286.id40884.diff
Thu, Dec 19, 9:58 AM
F3495773: D12286.id40923.diff
Thu, Dec 19, 9:58 AM
F3495764: D12286.id.diff
Thu, Dec 19, 9:58 AM
F3495763: D12286.diff
Thu, Dec 19, 9:58 AM
Unknown Object (File)
Sun, Dec 15, 4:04 AM
Unknown Object (File)
Wed, Nov 20, 1:29 PM
Subscribers

Details

Summary

part of ENG-7793
while testing I noticed that sometimes when I enter thread settings, a thread that does have an invite link appears to not have it. This is because we used to only fetch invite links when opening the community drawer, because this was the only place we were displaying them. Now I am also displaying them in thread settings, and the user may navigate to them without ever opening the drawer.
This is not the best solution, but this is a compromise we decided on when implementing invite links to avoid implementing updates for them

Test Plan

Tested that now when the user opens the app and navigates to thread settings immediately, the invite link is fetched

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.Jun 3 2024, 9:20 AM
tomek added inline comments.
native/chat/settings/thread-settings.react.js
1339–1349 ↗(On Diff #40884)

Maybe add a code comment explaining this.

This revision is now accepted and ready to land.Jun 4 2024, 1:10 AM