-
Notifications
You must be signed in to change notification settings - Fork 123
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
Repair Date Since #616
Repair Date Since #616
Conversation
Check if file has correct date
@@ -338,9 +338,10 @@ def download_videos( | |||
_LOGGER.info("No videos found on page %s. Exiting.", page) | |||
break | |||
|
|||
self._parse_downloaded_items(result, camera, path, delay, debug) | |||
self._parse_downloaded_items( | |||
result, camera, path, delay, debug, formatted_date) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where in the download_videos
method is formatted_date
set?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont see the problem. It works fine.
continue | ||
|
||
if created_at < formatted_date: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What type is formated_date
? Is it the same as created_at
? How do you ensure that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it will work implicite. It works as string also.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs within seven days. Thank you for your contributions. |
This issue is now being closed due to inactivity. |
It fixes the download issue. Only an idea. |
Check if file has correct date
Description:
Related issue (if applicable): fixes #604
Checklist:
tox
run successfully PR cannot be meged unless tests pass