-
Notifications
You must be signed in to change notification settings - Fork 307
Implemented Google Sign In Feature + MVC Structure #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| static Future login() async { | ||
| try { | ||
| var user = await _googleSignIn.signIn(); | ||
| print(user?.displayName!); | ||
| return user; | ||
| } catch (error) { | ||
| print(error); | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we know how to use this user and add it to Auth0 system?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jddeep Yes. Once this and Auth0 PR is merged, we will then create a common User model and initailize that with the loggedInUser info. Also, we'll encode our object into a JSON string and save it using get_storage. That will help us in keeping the user logged in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chandansgowda Are you talking about this PR - #15 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jddeep Yes. But that PR will have a lot of conflicts now. I have spoke to him over call. Once my PR gets merged, he will pull it and work on it again. Then once that gets merged, I will work on a common user model.
|
@jddeep can you please review this... so I can start working on next task... |
@jddeep As per your suggestions, I have implemented Google Sign In without using Webview/Firebase. As of now, I have used my own Google Cloud Console account for development. That has to be changed before deployment. I have also structured the files using MVC design pattern. The UI has to be implemented once the design is confirmed. For now, the required functionality is implemented.
Screen.Recording.2023-03-24.at.12.06.55.AM.mov
Closes #8
Closes #26