-
Notifications
You must be signed in to change notification settings - Fork 48
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
Feature: add custom date format #93
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this PR, I tested it locally and it works well.
The format string configuration may be cumbersome and error-prone without a reference and input validation handling. I made two suggestions inline, let me know what you think or have any questions.
There are also some typos in the settings, and I would suggest reverting the yarn.lock
changes.
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
Needs actually importing |
I have imported the I also include where the error is from in the title of the dialog box. This will help the user identify where the error is from when they refresh the browser. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @skngetich! Tested locally, works well.
This feature enable a user to add a custom date format.
i.e
yyy-MM-dd HH:mm:ss
result to2023-02-28 14:25:57
.dd/MM/YYY HH:mm:ss (O)
results to28/02/2023 14:25:57 (GMT +3)
,This is based on the date-fns format