-
Notifications
You must be signed in to change notification settings - Fork 15
JSON file 'data' converted to multiline human readable JSON using JsonLint #20
base: master
Are you sure you want to change the base?
Conversation
@@ -4,7 +4,8 @@ $(document).ready(function(){ | |||
var string = ""; | |||
var months = new Array("JAN","FEB","MAR","APR","MAY","JUN","JUL","AUG","SEP","OCT","NOV","DEC"); | |||
var events = data.objects; | |||
for (var i=events.length-1;i>=0;i--) { | |||
|
|||
for (var i=0;i<events.length;i++) { |
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.
I've a hunch that i
should be initialized at 1.
Also, please fix the indentation.
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.
Hey, can't get why i should be initialized at 1, 'events' array contains objects starting from events[0] to events[events.length-1], pls correct me if I'm wrong.
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.
r-
Indentation.
@debloper I think 0 would do
Please check the inline-comment.
|
Conflicts: data index.html
@debloper and all Can I have some comments on my last commit "JSON file 'data' converted to multiline human readable JSON" |
Fixes Pull Request #20 ; Recent -> Farthest (ordering)
No description provided.