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

README: correcting begin/end timestamp format #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

brianhanifin
Copy link

Correcting begintimestamp & endtimestamp format to match the Regular Expression test in the code.

I was following the mm/dd/yyyy hh:mm:ss format specified in the documentation, but imagedirector.delete_files kept failing. It wasn't until I took a screenshot of the error message that I figured out why. The regular expression test was failing because the code wants the day to appear before the month. I have corrected the documentation so that the next person could have better luck.

Error:

Failed to call service imagedirectory.delete_files. value 02/26/2021 00:00:00 does not match regular expression [0-3][0-9]/[0-1][0-9]/\d{4} [0-2]:[0-5][0-9]:[0-5][0-9] for dictionary value @ data['endtimestamp']

Correcting `begintimestamp` & `endtimestamp` format to match the Regular Expression test in the code.

`imagedirector.delete` kept failing, I was following the 'mm/dd/yyyy hh:mm:ss' format specified in the documentation. It wasn't until I took a screenshot of the error message that I figured out why. The regular expression test was failing because the code wants the day to appear before the month. I have corrected the documentation so that the next person could have better luck.

*Error: "Failed to call service imagedirectory.delete_files. value 02/26/2021 00:00:00 does not match regular expression [0-3][0-9]/[0-1][0-9]/\d{4} [0-2]:[0-5][0-9]:[0-5][0-9] for dictionary value @ data['endtimestamp']*
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.

1 participant