Skip to content

Remove Amplitude integration#17

Merged
cmyui merged 1 commit intomasterfrom
remove-amplitude
Mar 9, 2026
Merged

Remove Amplitude integration#17
cmyui merged 1 commit intomasterfrom
remove-amplitude

Conversation

@cmyui
Copy link
Member

@cmyui cmyui commented Mar 9, 2026

Summary

  • Remove TODO comments for Amplitude event logging in authenticate and initialize_password_reset
  • Remove unused [mypy-amplitude.*] section from mypy config

Amplitude was never actually integrated (no library, no adapter, no config) -- just placeholder TODOs and a preemptive mypy ignore.

Test plan

  • Verify service starts and authentication/password-reset flows work unchanged

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cmyui cmyui requested a review from infernalfire72 as a code owner March 9, 2026 11:01
@cmyui cmyui merged commit 1f299ae into master Mar 9, 2026
1 check passed
@cmyui cmyui deleted the remove-amplitude branch March 9, 2026 11:03
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