Skip to content
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

Fix moment.js deprecation warning #53

Merged
merged 1 commit into from
Oct 19, 2017
Merged

Fix moment.js deprecation warning #53

merged 1 commit into from
Oct 19, 2017

Conversation

vvv1559
Copy link
Contributor

@vvv1559 vvv1559 commented Oct 19, 2017

Fix issue #41
The problem: your data format MM/DD/YYYY is not ISO format. I changed working format to ISO-date format YYYY-DD-MM, and convert it only in Sidebar display

Also this change fix the problem of absent leading zeros in days and months with 1-digit

@@ -80,3 +80,5 @@ yarn-debug.log*
yarn-error.log*
.vscode/launch.json
package-lock.json

.idea/

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure it can't hurt!

what is this for, out of curiosity :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you are using IDE from JetBrains (I use WebStorm for web stack) - the IDE put own files into .idea path. It is unnecessary put those files into repository. This row hide path and subfiles for GIT version control system

@karismatic-megafauna karismatic-megafauna merged commit 0e7e583 into karismatic-megafauna:master Oct 19, 2017
@vvv1559 vvv1559 deleted the fix_moment_js branch October 19, 2017 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants