-
-
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
Handle Education Video fetching failure with content unavailable view. #84
Comments
I would take a look into this issue and possibly implement this along with issue #57, as I assume the solution to both issues should be nearly identical in their implementation of the |
Great idea! Fyi handling empty content has already been taken care of in the Heart Health view in a previous PR (I meant to close #57), so Education is the main place where this will be necessary. Feel free to adjust/add empty content handling in other views if you notice any inconsistencies! 🚀 |
…nce with issue StanfordBDHG#84: Update Education view to handle empty video collections and modification of VideoList to display message when no videos are available
…nce with issue StanfordBDHG#84: Update Education view to handle empty video collections and modification of VideoList to display message when no videos are available
Problem
Right now, if we fail to fetch the educational videos from firestore (for if there are no videos in firestore), we show an essentially empty view in the Education tab. Instead, we should show a
ContentUnavailableView
.Solution
Add an if statement that will either show the
VideoList
if theVideoManager
is not empty, and aContentUnavailableView
otherwise. Within theVideoList
, if avideoCollection
is empty (i.e. the section has no videos), consider not showing theVideoListSection
.Additional context
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: