Page MenuHomePhabricator
Differential D12647

[native] introduce barebones UI for ThreadSettingsNotifications
ClosedPublic
Actions

Authored by ginsu on Jul 3 2024, 1:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 25, 8:51 PM
Unknown Object (File)
Wed, Sep 25, 8:50 PM
Unknown Object (File)
Wed, Sep 25, 8:50 PM
Unknown Object (File)
Sun, Sep 8, 6:11 PM
Unknown Object (File)
Thu, Sep 5, 6:41 PM
Unknown Object (File)
Thu, Sep 5, 6:40 PM
Unknown Object (File)
Thu, Sep 5, 6:39 PM
Unknown Object (File)
Sep 4 2024, 6:25 AM
View All 58 Files
Subscribers
ashoat
tomek

Details

Reviewers
ashoat
inka
Commits
rCOMM769fcab17eac: [native] introduce barebones UI for ThreadSettingsNotifications
Summary

This diff introduces the barebones UI for ThreadSettingsNotifications. I know that there were visual changes requested in D12643, so this will also probably also need to be updated, but given that I am leaving soon, I want to get this out sooner rather than later

Depends on D12661

Test Plan

Please see the demo video below: (Please note the red block at the end is just something in my local stack and has not been included in any diffs)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu created this revision.Jul 3 2024, 1:11 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 3 2024, 1:11 AM
ginsu edited the summary of this revision. (Show Details)Jul 3 2024, 1:16 AM
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: ashoat, inka.
ginsu added a parent revision: D12646: [lib/web] introduce useThreadSettingsNotifications.
ginsu attached a referenced file: F2174800: Screen Recording 2024-07-03 at 4.13.35 AM.mov. (Show Details)
ginsu added a child revision: D12648: [native] introduce NotificationDescription.Jul 3 2024, 1:24 AM
Harbormaster completed remote builds in B30076: Diff 41930.Jul 3 2024, 1:27 AM
ginsu requested review of this revision.Jul 3 2024, 1:27 AM
inka accepted this revision.Jul 3 2024, 2:49 AM
This revision is now accepted and ready to land.Jul 3 2024, 2:49 AM
ashoat mentioned this in D12646: [lib/web] introduce useThreadSettingsNotifications.Jul 3 2024, 6:55 AM
ginsu removed a parent revision: D12646: [lib/web] introduce useThreadSettingsNotifications.Jul 3 2024, 3:06 PM
ginsu updated this revision to Diff 41993.Jul 3 2024, 9:18 PM
ginsu added a parent revision: D12646: [lib/web] introduce useThreadSettingsNotifications.

update

ginsu edited the test plan for this revision. (Show Details)Jul 3 2024, 9:20 PM
ginsu attached a referenced file: F2184503: Screen Recording 2024-07-04 at 12.18.06 AM.mov. (Show Details)
Harbormaster completed remote builds in B30122: Diff 41993.Jul 3 2024, 9:50 PM
ginsu edited the summary of this revision. (Show Details)Jul 3 2024, 10:09 PM
ginsu added a parent revision: D12661: [lib/native/web] factor out remaining shared logic + copy for thread notification settings.
ginsu removed a parent revision: D12646: [lib/web] introduce useThreadSettingsNotifications.
ginsu updated this revision to Diff 42050.Jul 4 2024, 10:59 AM

rebase before landing

This revision was landed with ongoing or failed builds.Jul 4 2024, 11:21 AM
Closed by commit rCOMM769fcab17eac: [native] introduce barebones UI for ThreadSettingsNotifications (authored by ginsu). · Explain Why
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM769fcab17eac: [native] introduce barebones UI for ThreadSettingsNotifications.
Harbormaster completed remote builds in B30159: Diff 42050.Jul 4 2024, 11:24 AM

Revision Contents
Changeset List

PathSize
native/
chat/
settings/
110 lines

Diff 42050

View Options

native/chat/settings/thread-settings-notifications.react.js

Loading...