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
Unknown Object (File)
Mon, Sep 9, 12:41 PM
Unknown Object (File)
Mon, Sep 9, 12:22 PM
Unknown Object (File)
Mon, Sep 9, 12:22 PM
Unknown Object (File)
Mon, Sep 9, 12:22 PM
Unknown Object (File)
Mon, Sep 9, 12:22 PM
Unknown Object (File)
Mon, Sep 9, 10:57 AM
Unknown Object (File)
Aug 28 2024, 6:49 AM
Unknown Object (File)
Aug 25 2024, 10:23 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
No Lint Coverage
Unit
No Test Coverage

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