prateek3255/scrroll-in

Shift Entire Codebase from ES5 to ES6.

Closed this issue · 7 comments

The codebase should be updated to use ES6 features. The current codebase is mixed with ES5 and ES6 syntax and practices.
The following needs to done

  1. Change all usages of 'var' keyword with 'let' or 'const'.
  2. Using promises instead of callbacks.

I am not seeing anything which can be changed to ES6 promises

Callback functions are being used in the Chrome API, we can change them to promises.

Hi, can i work on this?

Thanks for that @shikharsaxena98

Is this issue closed? Can I work on it?

@darkmatter18 & @wale-bant , I think @kohli6010 is working on this, else you can work on this.

@shikharsaxena98 I don't think Chrome API is available in Promises so as to maintain backward compatibility. I tried to move to promises but the extension keeps on throwing errors. Although there are a few places where var is used and function()s have been defined.

image

https://stackoverflow.com/questions/42191030/promise-support-for-chrome-extensions-api

Another thing that could be done is to use babel to compile the code to ES5