Skip to content

Conversation

@we-chintan-p001
Copy link
Collaborator

Task#1081 - [Team Mobile App] Create UI Widgets

Time invested: hh:mm format | Billable: 31:54 | Non-Billable: 00:00

Problem Statement
Refer https://drive.google.com/file/d/1Coy-QC2b5MgmxmBHQDP4dPWXBRdGgiRd/view and create UI elements for flutter

Describe The Fix/ Solution You Implemented

  • I've enhanced the code and done refactoring.
  • I've removed TODO: comments from the user model and Added constant widgets as helpers
  • I've moved some imp files from lib to lib/vaahextendflutter
  • I've added the default size and border in the app theme
  • I've updated env variable names and added methods to set EnvConfig and get Existing EnvConfig and
  • I've renamed Helpers to Alerts cause it contains toasts and dialogue boxes only, I've also modified the declaration type
  • I've updated the API init method
  • I've updated debug panel height and variable names
  • I've renamed view dir to views (to match it with models and controllers)
  • I've added a UI screen

Proof of your testing (Demo link or video links or image links)

Feature Merge Request Checklist

  • I have performed a self-review of my code
  • My code does follow industry standards
  • My code doesn't produce warnings/ errors for dart analyzer
  • New and existing tests pass locally with my changes (No tests are there as of now)
  • The code modified as part of this PR has been covered with tests
  • My code is properly formatted
  • I've followed the proper naming conventions
  • I have rebased the feature on the latest develop
  • I have run flutter run after the rebase
  • I have added new dependencies
  • I have read all the comments & notes in wireframe and verified that I did take care of that
  • verify that the UI must match with design & wireframe if available
  • I have updated the version/ build (x.x.x+xxx)
  • I've verified that the latest commit of develop exists in your feature branch after rebase.
  • My fix achieves 100% of what is required
  • Can enhance the solution in the future because better implementation could be made

@themodernpk themodernpk merged commit eea4d64 into develop Jan 13, 2023
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.

3 participants