Skip to content

Conversation

@chandansgowda
Copy link
Member

@chandansgowda chandansgowda commented Mar 23, 2023

@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

@chandansgowda chandansgowda changed the title Implemented Google Sign In Feature Implemented Google Sign In Feature + MVC Structure Mar 24, 2023
Comment on lines +10 to +18
static Future login() async {
try {
var user = await _googleSignIn.signIn();
print(user?.displayName!);
return user;
} catch (error) {
print(error);
}
}
Copy link
Member

@jddeep jddeep Mar 26, 2023

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?

Copy link
Member Author

@chandansgowda chandansgowda Mar 26, 2023

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.

Copy link
Member

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 ?

Copy link
Member Author

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.

@chandansgowda chandansgowda requested a review from jddeep March 26, 2023 07:26
@ViAsmit
Copy link

ViAsmit commented Mar 29, 2023

@jddeep can you please review this... so I can start working on next task...

@jddeep jddeep merged commit 357fe40 into AOSSIE-Org:master Mar 31, 2023
@chandansgowda chandansgowda deleted the google-signin branch March 31, 2023 13:47
@chandansgowda chandansgowda self-assigned this Aug 21, 2023
@chandansgowda chandansgowda added the enhancement New feature or request label Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create MVC Folder Structure Add Google Sign In with AuthO

3 participants