Skip to content

Commit

Permalink
fix: always record file views
Browse files Browse the repository at this point in the history
  • Loading branch information
Sean Yesmunt committed Sep 17, 2019
1 parent 1183f2d commit c52a566
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/ui/component/fileViewer/view.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,8 @@ export default function FileViewer(props: Props) {
: __('Loading');

const previousUri = usePrevious(uri);
const previousIsReadyToPlay = usePrevious(isReadyToPlay);
const isNewView = uri && previousUri !== uri && isPlaying;
const wasntReadyButNowItIs = isReadyToPlay && !previousIsReadyToPlay;
const [hasRecordedView, setHasRecordedView] = useState(false);

useEffect(() => {
if (isNewView) {
Expand All @@ -73,13 +72,14 @@ export default function FileViewer(props: Props) {
}, [isNewView, uri]);

useEffect(() => {
if (playTime && isReadyToPlay && wasntReadyButNowItIs) {
if (playTime && isReadyToPlay && !hasRecordedView) {
const timeToStart = Date.now() - playTime;
triggerAnalyticsView(uri, timeToStart);
claimRewards();
setPlayTime(null);
setHasRecordedView(false);
}
}, [setPlayTime, triggerAnalyticsView, isReadyToPlay, wasntReadyButNowItIs, playTime, uri, claimRewards]);
}, [setPlayTime, triggerAnalyticsView, isReadyToPlay, hasRecordedView, playTime, uri, claimRewards]);

useEffect(() => {
function handleResize() {
Expand Down

0 comments on commit c52a566

Please sign in to comment.