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

feat: vod display #686

Merged
merged 8 commits into from
Feb 12, 2025
Merged

Conversation

LevilkTheReal
Copy link
Contributor

@LevilkTheReal LevilkTheReal commented Jan 3, 2025

VOD display

Description

During file upload the user should see a preview about media related files that are html5 supported. Other files don't show any kind of preview.

Related Issues

https://solarpunk-test.atlassian.net/browse/SPDV-48

Changes Made

About the new feat

  • New FitVideo comp to wrap the styled player
  • Utils changes to support image and video types sepparately
  • Refactors to align to the new changes on display

About the fix

  • Paginated approach to check all of the tags for sync progress

How Has This Been Tested?

feat: I did manual tests. At the file uploader I tried out a couple of file types to see the previews.
fix: I checked the status of the uploaded files and the sync status. It should reach 100% after a certain amount of time.

Checklist

done - During file upload the user should see a preview about media related files that are html5 supported. Other files don't show any kind of preview.

done - In the Download tab, for the file preview show a video preview when the format is supported (Swarm hash > Find, or reached from the Download History).

done - Supported Image types:
'image/bmp', 'image/gif', 'image/vnd.microsoft.icon', 'image/jpeg', 'image/png', 'image/svg+xml', 'image/tiff', 'image/webp'

Screenshots (if applicable)

Preview feat:
Screenshot 2025-01-09 at 12 05 25

Fix for sync:
Screenshot 2025-01-09 at 12 18 43

Questions

  • Do we want to show the preview for downloaded files as well? For this we have to expicitly format the to a content type. Duable but might have security issues.

@aranyia aranyia requested review from ferencsarai and Cafe137 January 3, 2025 09:52
@LevilkTheReal
Copy link
Contributor Author

LevilkTheReal commented Jan 10, 2025

Update: New changes now allow the user to see the preview of html5 supported media files on the download page as well.

Screenshot 2025-01-10 at 18 53 25

@LevilkTheReal LevilkTheReal marked this pull request as ready for review January 10, 2025 17:53
@Cafe137
Copy link
Collaborator

Cafe137 commented Jan 13, 2025

Hello, nice PR!

I have only one comment.

export const META_FILE_NAME = '.swarmgatewaymeta.json'
export const PREVIEW_FILE_NAME = '.swarmgatewaypreview'

IIRC, Swarm Gateway and Bee Dashboard interops to some extent with the previews. I see the PREVIEW_FILE_NAME dropped the .jpeg extension to be more general.

This may affect the Swarm Gateway preview functionality. It would be good to clarify the role of the two products and their interoperability, and also define if .swarmgatewaymeta.json and .swarmgatewaypreview are a standard that we want to promote by supporting them in the official Swarm products.

Just an FYI.

@LevilkTheReal
Copy link
Contributor Author

Additional refactors to make type handling more general

@Cafe137 Cafe137 changed the title Feat/vod display feat: vod display Feb 12, 2025
@Cafe137 Cafe137 merged commit bcd3d50 into ethersphere:master Feb 12, 2025
2 checks passed
@LevilkTheReal LevilkTheReal deleted the feat/vod-display branch February 12, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants