UI: Remove theme control settings.

[AnthonyMarin]

Description

Fixes # (issue)
Addresses this card

generated by co-pilot
This pull request includes changes to simplify the SettingsOverlay component and remove support for preferred theme detection. The most notable updates involve commenting out unused components and code related to theme settings and night mode, as well as hardcoding the default theme to 'dark'.

Simplification of SettingsOverlay component:

• src/components/header/components/settingsDropdown/settingsOverlay/SettingsOverlay/SettingsOverlay.tsx: Removed unused imports (NightModeSettings, ThemePicker, and Alert) and commented out the corresponding panels in the component's JSX structure. This simplifies the UI by temporarily disabling the theme picker and night mode settings. [1] [2]

Removal of preferred theme detection:

• src/store/slices/themeSlice.ts: Commented out the preferredTheme logic that detects the user's system theme preference and hardcoded the default theme to 'dark'. This removes support for dynamic theme detection.

Testing

• Verify layouts are not broken
• Verify that light-mode is not available

[Maphikza]

The removal look great. but I am getting this error when I go to allowed pages on this branch, "
TypeError: Cannot read properties of undefined (reading 'requiresNpubManagement')
AllowedUsersLayout
src/components/allowed-users/layouts/AllowedUsersLayout.tsx:152
149 | }
150 |
151 | const modeConfig = MODE_CONFIGURATIONS[currentMode];

152 | const showNpubManagement = modeConfig.requiresNpubManagement ||
| ^ 153 | (localSettings.read_access.scope === 'allowed_users' || localSettings.write_access.scope === 'allowed_users');
154 | const showTiers = currentMode === 'paid' || currentMode === 'free' || currentMode === 'exclusive';
155 |
"

I my pull request this part is already handled I think we need to consolidate everything soon so that we can start testing what is outstanding.

[AnthonyMarin]

The removal look great. but I am getting this error when I go to allowed pages on this branch, " TypeError: Cannot read properties of undefined (reading 'requiresNpubManagement') AllowedUsersLayout src/components/allowed-users/layouts/AllowedUsersLayout.tsx:152 149 | } 150 | 151 | const modeConfig = MODE_CONFIGURATIONS[currentMode];

152 | const showNpubManagement = modeConfig.requiresNpubManagement ||
| ^ 153 | (localSettings.read_access.scope === 'allowed_users' || localSettings.write_access.scope === 'allowed_users');
154 | const showTiers = currentMode === 'paid' || currentMode === 'free' || currentMode === 'exclusive';
155 |
"

I my pull request this part is already handled I think we need to consolidate everything soon so that we can start testing what is outstanding.

Hm that's interesting. This really only touches a few unrelated lines and I don't get that error. Let me check it out further

[AnthonyMarin]

@Maphikza I also pulled main into my branch on my machine locally and I haven't received any TypeErrors 😕

[Maphikza]

Main is up to date it's just this branch when I was running it that gave me that error.

[Maphikza]

This works great mate.