-
Notifications
You must be signed in to change notification settings - Fork 15
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
Refactor common widgets #903
Conversation
uni/lib/view/common_widgets/pages_layouts/general/widgets/navigation_drawer.dart
Outdated
Show resolved
Hide resolved
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #903 +/- ##
======================================
+ Coverage 17% 17% +1%
======================================
Files 207 212 +5
Lines 6418 6434 +16
======================================
+ Hits 1037 1042 +5
- Misses 5381 5392 +11 |
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.
LGTM 🚀
e9ebcf2
to
bc38e49
Compare
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.
Just a quick nitpick
uni/lib/view/common_widgets/pages_layouts/general/widgets/refresh_state.dart
Show resolved
Hide resolved
The merge-base changed after approval.
43e6780
to
d2ed63a
Compare
uni/lib/view/common_widgets/widgets/request_failed_message.dart
Outdated
Show resolved
Hide resolved
uni/lib/view/common_widgets/widgets/request_failed_message.dart
Outdated
Show resolved
Hide resolved
@Process-ing any progress on this? |
cacdf3c
to
fb31eb6
Compare
uni/lib/view/common_widgets/pages_layouts/general/widgets/refresh_state.dart
Show resolved
Hide resolved
7fb2b31
to
7231975
Compare
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.
Great work 🚀, just have some concerns below:
uni/lib/view/common_widgets/pages_layouts/general/widgets/top_right_button.dart
Outdated
Show resolved
Hide resolved
baf12b4
to
51d60c4
Compare
Closes #900
Review checklist
whatsnew/whatsnew-pt-PT
changelog.md
with the change