1) A reference to the issue addressed by this pull-request:
closes #96
2) A description of the changes proposed in this PR:
Converts internal processing of event data to POSIXct, not Date.
Correctly sets the difftime unit to days (it was previously implicitly seconds).
Minor change to remove event data that occurs after the report cutoff datetime.
3) Add @mentions for the people who will review this PR:
None.
Please confirm that you have:
[x] Run devtools::test() and fixed all failing tests and warnings.
[x] Added suppressMessages() to any test message which breaks the test progress UI.
Please complete the following information:
1) A reference to the issue addressed by this pull-request:
closes #96
2) A description of the changes proposed in this PR:
Converts internal processing of event data to POSIXct, not Date.
Correctly sets the difftime unit to days (it was previously implicitly seconds). Minor change to remove event data that occurs after the report cutoff datetime.
3) Add @mentions for the people who will review this PR:
None.
Please confirm that you have:
devtools::test()and fixed all failing tests and warnings.suppressMessages()to any test message which breaks the test progress UI.Thank you for contributing to {SPCreporter}!!