Skip to content

Conversation

@Maphikza
Copy link
Member

Remove XNostr Verification from Advanced Settings

Overview

This PR removes the XNostr verification feature from the advanced settings panel, aligning the UI with backend changes that have already removed this functionality.

Changes

  • Removed XNostr panel from AdvancedSettingsLayout.tsx
  • Removed XNostr entry from the navigation in SettingsNavigation.tsx
  • Removed XNostr entry from the settings page in SettingsPage.tsx
  • Updated settings.types.ts to remove 'xnostr' from the SettingsGroupName type
  • Deleted XNostr component files:
    • Removed src/components/settings/panels/XNostrPanel.tsx
    • Removed src/components/settings/XNostrSettings.tsx

Motivation

The XNostr verification functionality has been removed from the backend, so these UI components are no longer needed. This change ensures the UI accurately reflects the current backend capabilities.

Testing

  • Verified that the advanced settings panel loads correctly without the XNostr section
  • Confirmed that the settings navigation no longer shows the XNostr option
  • Ensured that removing these components doesn't affect other settings sections

Maphikza added 2 commits May 21, 2025 14:58
- Removed XNostr panel from AdvancedSettingsLayout
- Removed XNostr entry from SettingsNavigation
- Removed XNostr entry from SettingsPage
- Updated settings.types.ts to remove xnostr from SettingsGroupName

This change aligns the UI with backend changes that have removed XNostr verification functionality.
- Removed XNostrPanel.tsx and XNostrSettings.tsx files
- These components are no longer needed since XNostr verification has been removed from the backend
@f7f376a1fcd0d0e11a10ed1b6577c9 f7f376a1fcd0d0e11a10ed1b6577c9 merged commit adf848e into main May 23, 2025
2 checks passed
@AnthonyMarin AnthonyMarin deleted the feat/remove-xnostr-verification branch June 26, 2025 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants