-
Notifications
You must be signed in to change notification settings - Fork 1
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
Zy/5 dashboard migrate indi app #234
Conversation
@gjwgit The
The vm test machine should be updated. |
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.
Overall look good, a few things to consider/fix
@zheyxu conflict here |
once Ashley's comments have been resolved I will review this @zheyxu |
Hi @junhaow1 @atangster, please check the updated pr. |
Pull Request Details
What issue does this PR address
DASHBOARD: Migrate Indi app dashboard to HealthPod
Link to associated issue: DASHBOARD: Migrate Indi app dashboard to HealthPod #5
Checklist
Complete the check-list below to ensure your branch is ready for PR.
Flutter Style Guide: https://survivor.togaware.com/gnulinux/flutter-style.html
make prep
orflutter analyze lib
)make qtest.tmp
screenshot included in issueFinalising
Once PR discussion is complete and reviewers have approved: