Revamp Project - Githubissues

Senior-Project-CSE-3213 / sp_flutter_app

iOS/Android app built with Flutter using MVVM architecture

3 stars 1 forks source link

Closed JustinASmith closed 4 years ago

JustinASmith commented 4 years ago

Fixes a few issues we had in configuration that prevented code signing capabilities.
Restructures directory structure.
Tidies up the use of AuthService.
- This is done through the AuthService being exposed via MultiProvider at the top of the widget tree and using Provider.of(context) in lower widgets (sign_in/register) to call AuthService methods.
Use simplified login/register widgets instead of ellaborate signin/register that was implemented before.

Resolves: #9

JustinASmith commented 4 years ago

@SamBoggs1337 I'll go over this when we meet today to help with any confusion you or anyone might have on this.

JustinASmith commented 4 years ago

Looks good, we may need to start consistently adding comments, for readability.

@SamBoggs1337 I agree. We can be on the look out for adding comments on unclear code

Schweinsei007 commented 4 years ago

Great, will be looking forward to learning more about it.