Skip to content

feat: firebase auth implementation#160

Merged
AnkushSarkar10 merged 1 commit intomainfrom
02-05-feat_firebase_auth_implementation
Mar 25, 2025
Merged

feat: firebase auth implementation#160
AnkushSarkar10 merged 1 commit intomainfrom
02-05-feat_firebase_auth_implementation

Conversation

@AnkushSarkar10
Copy link
Collaborator

@AnkushSarkar10 AnkushSarkar10 commented Feb 6, 2025

all the auth

Copy link
Collaborator Author

AnkushSarkar10 commented Feb 6, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@AnkushSarkar10 AnkushSarkar10 force-pushed the 02-05-feat_firebase_auth_implementation branch 2 times, most recently from d1efbd6 to ac559e6 Compare February 7, 2025 07:02
Copy link
Collaborator

@owengretzinger owengretzinger left a comment

Choose a reason for hiding this comment

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

Can we use MMKV? And then we should be able to get rid of all the async stuff unless it depends on the server.

If it doesn't get rid of all the async, can we create a hook so that we don't need the loading useStates everywhere?

Also fix type errors

@AnkushSarkar10 AnkushSarkar10 marked this pull request as ready for review February 9, 2025 04:31
@graphite-app graphite-app bot requested a review from owengretzinger February 9, 2025 04:32
@graphite-app
Copy link

graphite-app bot commented Feb 9, 2025

Graphite Automations

"Request reviewers once CI passes" took an action on this PR • (02/09/25)

1 reviewer was added to this PR based on Owen Gretzinger's automation.

@AnkushSarkar10 AnkushSarkar10 force-pushed the 02-05-feat_firebase_auth_implementation branch from ac559e6 to 4483bef Compare February 9, 2025 04:46
@AnkushSarkar10 AnkushSarkar10 force-pushed the 02-05-feat_firebase_auth_implementation branch from 4483bef to cc0d367 Compare February 18, 2025 17:55
Copy link
Collaborator

Also can you change the default pfp to an actual placeholder pfp

@AnkushSarkar10 AnkushSarkar10 force-pushed the 02-05-feat_firebase_auth_implementation branch from cc0d367 to 638cb92 Compare March 1, 2025 21:01
@AnkushSarkar10 AnkushSarkar10 merged commit 4c6d2bd into main Mar 25, 2025
2 checks passed
Copy link
Collaborator Author

Merge activity

  • Mar 25, 12:12 AM EDT: A user merged this pull request with Graphite.

@AnkushSarkar10 AnkushSarkar10 deleted the 02-05-feat_firebase_auth_implementation branch March 25, 2025 04:12
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.

2 participants