Review of "Stocks Extension" version 22

Details Page Preview

Stocks Extension brings stock quotes to your GNOME Shell Panel

Extension Homepage
https://github.com/internetstaff/stocks-extension

No comments.

Diff Against

Files

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

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Previous Reviews on this Version

JustPerfection rejected
1. Timeout should be remove on destroy (line 82 components/stocks/menuStockTicker.js). That callback can get triggered after disable. 2.Forgot to null out `stocksMenu` in disable. Add this to disable (extension.js): ```js stocksMenu = null; ``` 3. `SETTINGS_SCHEMA_DOMAIN` is unused. Remove it from `helpers/settings.js` (line 17).