Skip to content
This repository has been archived by the owner on Dec 2, 2019. It is now read-only.

Implement timestamp option for ios #21

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

n1ru4l
Copy link
Contributor

@n1ru4l n1ru4l commented Apr 12, 2018

Implements #18

Code sample:

import RNThumbnail from 'react-native-thumbnail';
const timestampInSeconds = 3
RNThumbnail.get(filepath, { timestamp: timestampInSeconds }).then((result) => {
  console.log(result.path); // thumbnail path
})

@n1ru4l n1ru4l force-pushed the feature-ios-thumbnail-timestamp branch from 7e6358c to 961bf0f Compare April 12, 2018 12:54
@phuochau
Copy link
Owner

@n1ru4l I think the code example is wrong? The timestamp should be an option of get function

@n1ru4l
Copy link
Contributor Author

n1ru4l commented Oct 15, 2018

@phuochau I updated the example, however the code is fully functional

@watadarkstar
Copy link

@n1ru4l What would happen if that frame / timestamp doesn't exist?

@n1ru4l
Copy link
Contributor Author

n1ru4l commented Feb 18, 2019

@watadarkstar You will receive the last frame if the timestamp exceeds the maximum duration of the video.

@phuochau Is there anything further you need to be added before merging this?

Offtopic:

@phuochau Have you considered moving this repository over to react-native-community or adding additional maintainers? This could improve the quality and development of this library significantly.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants