search

ddrdushy / FCC-Status

1 stars 7 forks source link

Removed time complexcity #16

Closed ddrdushy closed 7 years ago

ddrdushy commented 7 years ago

code redesigned for making async request ajax calls.

kgashok commented 7 years ago
kgashok commented 7 years ago

Dushyanth,

The JS FCC CamperBot app is really fast now! Congratulations on the improved performance.

kgisl/FCC-Status deploys automatically to Heroku at https://kgfcc.herokuapp.com/

% ashok

On 27 December 2016 at 00:15, ddr dushy notifications@github.com wrote:

code redesigned for making async request ajax calls.

You can view, comment on, or merge this pull request online at:

https://github.com/ddrdushy/FCC-Status/pull/16 Commit Summary

  • Formatted data.json into multiple lines for better readability
  • Renamed the README file; add some more comments
  • Refactored code from index.html into .js and .css files
  • Added the the meta tags DOCTYPE and character encoding, Issue #10
  • Update gBot.js
  • Merge pull request #1 from kgisl/kgashok-patch-1
  • Fix indendation using Sublime Text 3
  • Added PHP file for deploying to Heroku
  • Added "unresponsive error" screenshot, Issue #2
  • added app and static files
  • empty
  • Added app.json and static.json to deploy on Heroku, Issue #6
  • minor edits
  • minor change in CSS
  • Merge branch 'master' of https://github.com/kgisl/FCC-Status
  • Formatted for better readability
  • Added back the PHP file as a legacy asset
  • Renamed image file name to avoid pull problems as in https://github.com/ddrdushy/FCC-Status/issues/13 https://github.com/ddrdushy/FCC-Status/issues/13
  • Added Grunt and Travis for JS Linting
  • Added Travis status image
  • Modifications to pass lint errors
  • the JS file now lives in ./src/js
  • Fixed jshint issues except in Gruntfile.js
  • Changed indentation to 2; Resolved #7 and ddrdushy/FCC-Status/#15
  • Update gBot.js
  • Merge pull request #8 from kgisl/kgashok-patch-1
  • Update gBot.js
  • Update style.css
  • moved line into the right place
  • Merge pull request #13 from kgisl/origin/kgashok-patch-4
  • Use KG colours for highlighting!
  • Added qunit testing
  • modified Gruntfile.js to incorporate qunit tests
  • Gruntfile.js also lint ready; Resolved #14
  • Introduced opacity for the highlighting
  • Update style.css
  • Merge pull request #17 from kgisl/kgashok-patch-1
  • Update style.css
  • Merge pull request #18 from kgisl/kgashok-patch-1
  • Added automated testing using grunt watch
  • Change made to inform CI and CD and Auto-testing; Resolves #19
  • Revert back to not include watch; to fix Travis fail
  • Use the right version for watch
  • changed version numbers to fix Travis fail
  • revert back to earlier
  • avoid linting gruntfile for now
  • included gruntfile back
  • removed watch
  • added watch back
  • added phantom back
  • removed phantomJS ref
  • Qunit coverage also possible; sample Bot code tested; resolved #20
  • Used jsdevenv Gruntfile.js and node_modules dir; Resolved #20
  • Removed all lint errors; reverted back to tab space 4
  • gBot.js location altered in index.html file
  • Coverage tests were failing; commented out the require line
  • lint-free'ing getBrownies
  • added jquery to get tests to pass; coverage still failing
  • Gruntfile fix to make Travis build pass
  • minor edits
  • Changed node version; made async test
  • Removed test to make Travis CI pass
  • Included coverage
  • included test back to check on Travis CI
  • added test back in
  • moving back to previous state
  • Lint error fixed
  • removed test to make Travis CI pass
  • Added back Ajax test to see if it fails Travis CI
  • fixed Lint error
  • changed from asynTest to Test
  • Update gBot.js
  • Merge pull request #24 from kgisl/kgashok-patch-1
  • Update index.html
  • Add files via upload
  • Update gBot.js
  • Merge pull request #25 from kgisl/kgashok-patch-1
  • Update gBot.js
  • Merge pull request #26 from kgisl/kgashok-patch-1
  • Update Gruntfile.js
  • Merge pull request #28 from kgisl/kgashok-patch-1
  • Update Gruntfile.js
  • Update Gruntfile.js
  • Account skiped
  • Update gBot.js
  • Update getBrownies.test.js
  • fixed test case
  • Merge branch 'master' of https://github.com/kgisl/FCC-Status
  • Merge pull request #30 from kgisl/kgashok-patch-2
  • Update README.md
  • Merge pull request #31 from kgisl/kgashok-patch-1
  • added shrinkwrap
  • Merge branch 'master' of https://github.com/kgisl/FCC-Status
  • Trying coverage on snap-ci
  • switched back to 'test'
  • trying coverage on snap-ci
  • removing node_modules folder
  • replaced coverage with test
  • Added node_modules to .gitignore
  • Removed coverage
  • moved shrinkwrap out to backup
  • added coverage
  • added test back in

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ddrdushy/FCC-Status/pull/16, or mute the thread https://github.com/notifications/unsubscribe-auth/ACSbIqGC2xhIK33vKA_Z_zu-BDYS-_rdks5rMAs4gaJpZM4LV4Ta .