-
Notifications
You must be signed in to change notification settings - Fork 19
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
Schedule card #1293
base: develop
Are you sure you want to change the base?
Schedule card #1293
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #1293 +/- ##
=======================================
Coverage 12% 12%
=======================================
Files 264 264
Lines 7238 7238
=======================================
Hits 803 803
Misses 6435 6435 |
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 tooltip missing in the clock, but we can come back latter.
✅ 🛫
Also, for every text, we should assign a theme text style. This way we ensure consistent in the all app |
4224cfe
to
a102c11
Compare
// final BadgeColors badgeColor; | ||
final bool isActive; | ||
final String? teacherName; | ||
final String? teacherPhoto; |
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.
teacherPhoto
is not being used
Closes #1273
Review checklist
whatsnew/whatsnew-pt-PT
changelog.md
with the change