Skip to content

Conversation

@maxschwinghammer
Copy link
Collaborator

Summary

  • Avoid scheduling notifications when no user is logged in
  • Redirect unauthenticated users to the sign-in screen on launch

Testing

  • ./gradlew test (fails: Gradle daemon started but tests did not run due to environment constraints)

https://chatgpt.com/codex/tasks/task_b_68a70fa0fba88327887a7e3ad8acfa98

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements null-safe Firebase authentication handling to prevent crashes and ensure proper user flow. The changes ensure notifications are only scheduled for authenticated users and redirect unauthenticated users to the sign-in screen.

  • Move Firebase auth check from class property to local variable for thread safety
  • Add null check before scheduling notifications to prevent crashes
  • Implement redirect logic for unauthenticated users using LaunchedEffect

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@maxschwinghammer
Copy link
Collaborator Author

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link

Copilot AI commented Nov 10, 2025

@maxschwinghammer I've opened a new pull request, #231, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants