HomePhabricator
Diffusion Comm 6eca198adcf6

[web] Rename `NotificationsOption` -> `EnumSettingsOption`

Description

[web] Rename NotificationsOption -> EnumSettingsOption

Summary:
We want to reuse the (previously known as) NotificationsOption component for the "Privacy" tab of ThreadSettingsModal.

As a first step, we rename NotificationsOption to more generic EnumSettingsOption (open to alternative names). In later diffs will remove "Notifications-specific" parts of EnumSettingsOption and make it more generic.

Made sure to rename component, rename file, and fix imports.


Depends on D4044

Test Plan: Opened NotificationsModal and it continues to look/work as expected. Also flow/eslint.

Reviewers: def-au1t, palys-swm, benschac

Reviewed By: palys-swm, benschac

Subscribers: ashoat, Adrian, yayabosh

Differential Revision: https://phabricator.ashoat.com/D4045

Details

Provenance
atulAuthored on May 15 2022, 9:39 AM
Reviewer
tomek
Differential Revision
D4045: [web] Rename `NotificationsOption` -> `EnumSettingsOption`
Parents
rCOMM387806bb8d38: [web] Cut extraneous/unused styles from `thread-settings-privacy-tab.css`
Branches
Unknown
Tags
Unknown