Skip to content
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

MVVM implementation for Profile Page (profile_page.dart). #766

Closed
rutvik11062000 opened this issue May 7, 2021 · 1 comment · Fixed by #790
Closed

MVVM implementation for Profile Page (profile_page.dart). #766

rutvik11062000 opened this issue May 7, 2021 · 1 comment · Fixed by #790
Assignees

Comments

@rutvik11062000
Copy link
Contributor

Is your feature request related to a problem? Please describe.
This is the child issue of #758
To implement MVVM architecture on profile_page.dart file.

Describe the solution you'd like
See #760 for in-depth details about the implementation.
For Sample see the PR changes #759

Additional context
Other things to take into consideration:

  1. Make sure all the files that you are either updating or creating must be under 300 lines.
  2. If there are multiple same widget with only few modification extract it as a method and give only change variable as a parameter.
  3. Make the code as concise as possible.
  4. Kindly add setup locator in the profile_page_test.dart otherwise travis build will fail.
    image
@rutvik11062000
Copy link
Contributor Author

@Sagar2366, @palisadoes I would like to work on this.

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 a pull request may close this issue.

2 participants