Review of "Gnoming Profiles" version 2

Details Page Preview

Sync gsettings and user config files to GitHub with binary-safe wallpaper syncing and ETag-based polling for ultra-efficient bandwidth usage

Extension Homepage
https://github.com/gavindi/gnoming-profiles

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Version Status
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. You should remove the timeout before creating a new one since the function can be called multiple times (line 268 `lib/SyncManager.js`) 2. You forgot to remove the timeout on destroy (line 300 `lib/Utils.js`). 3. What's the use case of `debounce` and `throttle` anyway? Those functions are never used in your code. So why not just removing them?