Skip to content

Clean up Settings: remove Discussions, update contact, remove dead code#32

Merged
dadachi merged 1 commit intomainfrom
fix/remove-discussions-from-settings
Apr 1, 2026
Merged

Clean up Settings: remove Discussions, update contact, remove dead code#32
dadachi merged 1 commit intomainfrom
fix/remove-discussions-from-settings

Conversation

@dadachi
Copy link
Copy Markdown
Contributor

@dadachi dadachi commented Apr 1, 2026

Summary

  • Remove Discussions link from Settings screen
  • Replace MFMailComposeViewController with mailto: URL for contact (includes device info)
  • Replace App Store review link with StoreKit requestReview()
  • Delete MailView.swift and GlassButtonStyle.swift (unused)
  • Clean up unused constants

Test plan

  • make lint passes
  • Settings screen renders correctly
  • Contact link opens default mail app with device info
  • Rate button triggers native review prompt

🤖 Generated with Claude Code

Replace MFMailComposeViewController with mailto URL for contact.
Replace App Store review link with StoreKit requestReview().
Remove Discussions link, MailView.swift, GlassButtonStyle.swift,
and unused constants.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dadachi dadachi force-pushed the fix/remove-discussions-from-settings branch from afe298a to 9378db7 Compare April 1, 2026 07:30
@dadachi dadachi merged commit 9b8a627 into main Apr 1, 2026
2 checks passed
@dadachi dadachi deleted the fix/remove-discussions-from-settings branch April 1, 2026 08:24
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.

1 participant