-
Notifications
You must be signed in to change notification settings - Fork 59
Conversation
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.
Based on the messages and the UI shown in the PR I approve this.
However, I have not tested it. @anitab-org/qa-team can anyone here test this, please.
@@ -132,3 +119,17 @@ class _RequestsPageState extends State<RequestsPage> { | |||
); | |||
} | |||
} | |||
|
|||
class NoRequestHelper extends StatelessWidget { |
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 job.
I'd change NoRequestHelper
-> NoRequestsInfo
or NoRequestsText
and we're good to go (because it's not really a helper)!
🎉💥
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.
Done. :)
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.
:D
The iOS build is running since a hour and a half 😨 , I'll rerun it |
@isabelcosta can you approve this again? made some changes |
@anitab-org/qa-team can anyone test this? or @bartekpacia @snitin315 @shivaylamba |
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.
It's working @isabelcosta !! great job @techno-disaster 🎉 👍
thanks for testing@robotjellyzone |
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.
Sounds good then :)
Thank you @robotjellyzone for testing it!
Description
This block of code adds the helper text "you have {x} mentorship requests" to the requests page
Fixes #68
Flutter Channel:
Type of Change:
Delete irrelevant options.
Code/Quality Assurance Only
How Has This Been Tested?
On my physical device
Checklist:
Delete irrelevant options.
Code/Quality Assurance Only