Repository: Automattic/pocket-casts-ios
File: podcasts/NotificationsViewController.swift (Swift)

Code before:
                return L10n.notificationsDailyReminders
            case .newFeaturesAndTips:
                return L10n.notificationsNewFeaturesTips
            case .pocketCastsOffers:
                return L10n.notificationsPocketCastOffers
            }
        }

        var value: Bool {
            switch self {
            case .newEpisodes:
                return false
            case .podcastsChosen:
                return false
            case .appBadges:
                return false
            case .trendingRecommendations:
                return false
            case .dailyReminders:
                return Settings.notificationsOnboardingTips
            case .newFeaturesAndTips:
                return false
            case .pocketCastsOffers:
                return false
            }
        }
    }

    @IBOutlet var settingsTable: UITableView! {
        didSet {
            settingsTable.register(UINib(nibName: "SwitchCell", bundle: nil), forCellReuseIdentifier: switchCellId)
            settingsTable.register(UINib(nibName: "DisclosureCell", bundle: nil), forCellReuseIdentifier: disclosureCellId)
        }
    }

    override func viewDidLoad() {
        super.viewDidLoad()

        title = L10n.settingsNotifications
        NotificationCenter.default.addObserver(self, selector: #selector(podcastUpdated(_:)), name: Constants.Notifications.podcastUpdated, object: nil)

        Analytics.track(.settingsNotificationsShown)
    }

    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)

        settingsTable.reloadData()
    }

    func numberOfSections(in tableView: UITableView) -> Int {

Review comment (suggestion):
I think collapsing the falses would make skimming a bit easier.

```suggestion
        var value: Bool {
        case .dailyReminders:
            return Settings.notificationsOnboardingTips
        case .newEpisodes,
                 .podcastsChosen, 
                 .appBadges,
                 .trendingRecommendations,
                 .newFeaturesAndTips,
                 .pocketCastsOffers:
            return false
            }
        }
```

Code after:
                return L10n.notificationsDailyReminders
            case .newFeaturesAndTips:
                return L10n.notificationsNewFeaturesTips
            case .pocketCastsOffers:
                return L10n.notificationsPocketCastOffers
            }
        }

        var value: Bool {
            switch self {
            case .dailyReminders:
                return Settings.notificationsOnboardingTips
            case .newEpisodes,
                 .podcastsChosen,
                 .appBadges,
                 .trendingRecommendations,
                 .newFeaturesAndTips,
                 .pocketCastsOffers:
                return false
            }
        }
    }

    @IBOutlet var settingsTable: UITableView! {
        didSet {
            settingsTable.register(UINib(nibName: "SwitchCell", bundle: nil), forCellReuseIdentifier: switchCellId)
            settingsTable.register(UINib(nibName: "DisclosureCell", bundle: nil), forCellReuseIdentifier: disclosureCellId)
        }
    }

    override func viewDidLoad() {
        super.viewDidLoad()

        title = L10n.settingsNotifications
        NotificationCenter.default.addObserver(self, selector: #selector(podcastUpdated(_:)), name: Constants.Notifications.podcastUpdated, object: nil)

        Analytics.track(.settingsNotificationsShown)
    }

    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)

        settingsTable.reloadData()
    }

    func numberOfSections(in tableView: UITableView) -> Int {
        return FeatureFlag.notificationsRevamp.enabled ? 3 : 1
    }

    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        guard let sectionType = Section(rawValue: section) else {