Skip to content
This repository has been archived by the owner on Feb 15, 2023. It is now read-only.

Comments

@Fleury14
Copy link
Collaborator

Expected Behavior/Desired Use Case

Order Time should be correctly formatted in hh:mm:ss format.

Actual/Current Behavior

Order Time does not zero-pad minute and seconds correctly, so it will sometimes show invalid time formats such as this example:

Screen Shot 2020-02-11 at 8 03 23 PM

Steps to Reproduce the Problem/Implement this feature

  1. Create an order when the current time minutes or seconds is less than 10.

Learning

This is a common type of problem when time parsing is done manually instead of using built-in time formatting tools. Research best-practices for displaying time in the current stack and apply those best practices to this project.

bip12 referenced this issue in bip12/react-challenge-project-jan-2022 Jan 9, 2022
@T145 T145 linked a pull request Jan 13, 2022 that will close this issue
logan-hernandez referenced this issue in logan-hernandez/react-challenge-project-jan-2022 Apr 15, 2022
@zacharynoel zacharynoel linked a pull request Jun 26, 2022 that will close this issue
1 task
JimOtermat2 referenced this issue in JimOtermat2/bitwise-react-challenge-project Aug 6, 2022
Casts to string, then uses the padStart function to make sure the
minute and second show up as two digits

Fixes #2
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.