Skip to content
This repository has been archived by the owner on Apr 3, 2020. It is now read-only.

Adjust time method needs to deal with rolling over the date also #61

Open
pavulon18 opened this issue May 5, 2018 · 0 comments
Open
Assignees
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@pavulon18
Copy link
Owner

Describe the bug
A clear and concise description of what the bug is.

currently, when a time is between 2346 and 2350, the method sets the minutes to "00" and adds one to the hour, which in this case would make it "24". In this case, the 24 needs to be made "00" and the day needs +1. Perhaps I am not handling the time properly. I'll need to look into this.

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@pavulon18 pavulon18 self-assigned this May 5, 2018
@pavulon18 pavulon18 added bug Something isn't working help wanted Extra attention is needed good first issue Good for newcomers labels May 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant