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

show durations and published date on the video thumbnails #48

Closed
wants to merge 16 commits into from

Conversation

aninda052
Copy link
Contributor

@aninda052 aninda052 commented Jul 27, 2023

This pr will close multiple issues -

close #28

  • add ffprobe package for getting video metadata
  • add @ffprobe-installer/ffprobe for platform independent binary installer
  • add durations filed in videos collection to store video durations in second
  • get video durations in seconds using ffprobe and store that while storing video info in db
  • use react-moment package to format(HH:mm:ss) video durations.

close #29

  • shows recordingDate as published date in video thumbnails.
  • use react-moment package to format(DD/MM/yyyy) recordingDate.

@foyzulkarim
Copy link
Owner

thanks for the pull request brother. I would like to request you to allow me some time (2 weeks) to complete my current refactoring on the server side (making genric repo thing). then i can accept this PR. Thanks 🙏

@foyzulkarim
Copy link
Owner

@aninda052 please have a look
#51

aninda052 added a commit to aninda052/mern-video-streaming that referenced this pull request Aug 2, 2023
@foyzulkarim
Copy link
Owner

@aninda052 please resolve the conflicts.. thanks 👍

@aninda052
Copy link
Contributor Author

@aninda052 please resolve the conflicts.. thanks +1

@foyzulkarim conflict resolve

@foyzulkarim
Copy link
Owner

what is the difference between this work and #47 ?

either use different code in different PR, or close either of the PR please.

@aninda052
Copy link
Contributor Author

aninda052 commented Aug 6, 2023

closing this pr as i already merge feature/video-durations (#48) into feature/update-video-status (#47)

@foyzulkarim foyzulkarim self-requested a review August 21, 2023 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

show published date using date library show durations on the video thumbnails
2 participants