Cookie-Manager icon indicating copy to clipboard operation
Cookie-Manager copied to clipboard

[Documentation Missing] Add JSDoc

Open vinitshahdeo opened this issue 5 years ago • 1 comments

There are many functions in background.js which are using chrome.cookies API. Adding a short description before each function will make the code more readable.

For example,

/*
 *
 * @description event listener which notifies whenever cookie's properties are updated
 */
function onCookieChanged(){
  chrome.cookies.onChanged.addListener(function(cookies){
      console.log("cookies are being changed ", cookies.cookie.domain);
      console.log(cookies);
  });
}

  • Add JSDoc for each function.
  • Do not make any changes to other files.
  • Write assumptions if any.

Please refer to JSDoc and chrome.cookies API.

vinitshahdeo avatar Oct 11 '19 14:10 vinitshahdeo

Happy to grab this one.

hotdang-ca avatar Oct 11 '19 15:10 hotdang-ca