Skip to content

Comments

Add MyTBA profile photo, overflow menu, and FCM registration#1029

Closed
gregmarra wants to merge 1 commit intomainfrom
android-fcm-registration
Closed

Add MyTBA profile photo, overflow menu, and FCM registration#1029
gregmarra wants to merge 1 commit intomainfrom
android-fcm-registration

Conversation

@gregmarra
Copy link
Contributor

Summary

  • Register device as android-fcm client type (instead of android) for push notification support
  • Show Google profile photo as circular avatar on MyTBA screen, with person-icon fallback for loading/error/null states
  • Move "Sign out" behind overflow (MoreVert) dropdown menu
  • Add confirmation dialog before signing out ("You will no longer receive notifications for your favorites and subscriptions.")

Test plan

  • Sign in to MyTBA — verify profile photo loads (or person icon shows as fallback)
  • Tap overflow menu — verify "Sign out" option appears
  • Tap "Sign out" — verify confirmation dialog appears with Cancel/Sign out buttons
  • Tap Cancel — verify dialog dismisses without signing out
  • Tap Sign out — verify user is signed out
  • Verify FCM device registration sends android-fcm as operating system

🤖 Generated with Claude Code

Show the user's Google profile photo as a circular avatar (with a
person-icon fallback), move sign-out behind a MoreVert dropdown menu,
and require confirmation via an AlertDialog before signing out.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gregmarra gregmarra force-pushed the android-fcm-registration branch from 6574cc7 to 35e7868 Compare February 13, 2026 00:55
@gregmarra
Copy link
Contributor Author

Recreating with only the relevant commit (was also including the FCM registration commit).

@gregmarra gregmarra closed this Feb 13, 2026
@gregmarra gregmarra deleted the android-fcm-registration branch February 13, 2026 00:55
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