Skip to content
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

Remote Content Browsing: Update Content Renderer for downloading and streaming #9856

Closed
2 tasks
marcellamaki opened this issue Nov 21, 2022 · 2 comments
Closed
2 tasks
Assignees
Labels
DEV: frontend P0 - critical Priority: Release blocker or regression

Comments

@marcellamaki
Copy link
Member

marcellamaki commented Nov 21, 2022

Overview

This issue is a front-end issue to ensure that the content renderer correctly manages download and streaming states. The backend for this remote content browsing work is managed separately.

This implementation depends on both the DownloadRequests model, which indicates the downloads on the user's device as well as the useDevices composable which supports remote content browsing.

Specifications

Figma

Screenshot 2023-02-13 at 5 11 44 PM

Acceptance Criteria

  • If a user is browsing remote content, just "watching" the content, this should display a streaming status and the 'bookmark action' should be replaced with a 'download' action
  • if the user has downloaded the content to their device, they should see that it is downloaded, and still be able to bookmark the content

References

@MisRob
Copy link
Member

MisRob commented Apr 26, 2023

Related PR #10558. Some logic is mocked there though. This issue can be closed after #10564 is resolved.

@marcellamaki
Copy link
Member Author

Confirming that after manual QA including the team session on Friday, the download icon is replaced by the bookmark icon, once the download has completed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DEV: frontend P0 - critical Priority: Release blocker or regression
Projects
None yet
Development

No branches or pull requests

2 participants