Skip to content
This repository has been archived by the owner on Jun 8, 2022. It is now read-only.

Sanity Check #10

Closed
rad10 opened this issue Jan 9, 2020 · 1 comment · Fixed by #19
Closed

Sanity Check #10

rad10 opened this issue Jan 9, 2020 · 1 comment · Fixed by #19
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed

Comments

@rad10
Copy link
Owner

rad10 commented Jan 9, 2020

It might be worth looking into implementing an additional phase where I take the text results and to an additional scan to see if it makes logical sense.
Some examples to consider:

  • A persons name should never appear more than once on the same page for the same day.
  • The time that someone clocks out should never be less than the time that they clock in.
  • The time that the person stays should never be more than 12 hours.

How to respond to issues like these is for contemplation in future.

@rad10 rad10 added bug Something isn't working enhancement New feature or request labels Jan 9, 2020
@rad10 rad10 added this to the Glitchin and Fixin milestone Jan 9, 2020
@rad10
Copy link
Owner Author

rad10 commented Mar 30, 2020

Things become far more complicated when dealing with 12 hour time with no addition hinting at AM or PM. People appearing more than once has been taken care of

@rad10 rad10 added the help wanted Extra attention is needed label Mar 30, 2020
@rad10 rad10 mentioned this issue Mar 30, 2020
@rad10 rad10 linked a pull request Mar 30, 2020 that will close this issue
@rad10 rad10 closed this as completed in #19 Mar 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant